-
Notifications
You must be signed in to change notification settings - Fork 1.5k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fixed #12348 - fixed missing analyzer information for markup files #6952
Conversation
lib/cppcheck.cpp
Outdated
// TODO: set analyzer information | ||
|
||
// TODO: how to get the proper tokenlist to generate the proper hash? | ||
const std::size_t hash = time(nullptr); // calculateHash(tokenizer.list, mSettings); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This requires a simplecpp::TokenList
which we do not create in this context.
This is a hack so the hash is different every time.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The Preprocessor::calculateHash is pretty simple it should be possible to write a similar function that uses tokenizer.list
.. as shown in the comment?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I can have a look.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I now generate a hash from the proper inputs.
178381c
to
ee3d733
Compare
No description provided.