Skip to content

Fix #13889 (cached misra results are shown after removing --premium=misra-c-2012 option) #7594

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.

Sign up for GitHub

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

Merged
danmar merged 10 commits into from
Jun 17, 2025
+65 −14
Merged

Fix #13889 (cached misra results are shown after removing --premium=misra-c-2012 option) #7594

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
5ff2826
Add addons and premium args to hhash function
glankk Jun 13, 2025
1322092
Add test
glankk Jun 16, 2025
dd642df
Remove leftover NullCmdLineLogger
glankk Jun 16, 2025
9ae2624
Run dmake
glankk Jun 16, 2025
82f564e
Fix documentation error
glankk Jun 16, 2025
3cf2f1e
Remove unused TokenList forward declaration
glankk Jun 16, 2025
8e29143
Use const
glankk Jun 16, 2025
ddb384f
Revert "Remove unused TokenList forward declaration"
glankk Jun 16, 2025
3e0e3ff
Make calculateHash const
glankk Jun 16, 2025
c5a354b
Add suppression
glankk Jun 17, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions lib/cppcheck.cpp
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -861,6 +861,11 @@ static std::size_t calculateHash(const Preprocessor& preprocessor, const simplec
toolinfo << (settings.severity.isEnabled(Severity::information) ? 'i' : ' ');
toolinfo << settings.userDefines;
toolinfo << std::to_string(static_cast<std::uint8_t>(settings.checkLevel));
for (const auto &a : settings.addonInfos) {
toolinfo << a.name;
toolinfo << a.args;
}
toolinfo << settings.premiumArgs;
// TODO: do we need to add more options?
supprs.nomsg.dump(toolinfo);
return preprocessor.calculateHash(tokens, toolinfo.str());
Expand Down
Loading