Skip to content

fixed #14585 - store all errors in AnalyzerInformation even if suppressed - #8267

Merged
firewave merged 1 commit into
cppcheck-opensource:mainfrom
firewave:ainfo-suppr
Mar 15, 2026
Merged

fixed #14585 - store all errors in AnalyzerInformation even if suppressed#8267
firewave merged 1 commit into
cppcheck-opensource:mainfrom
firewave:ainfo-suppr

Conversation

@firewave

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave

Copy link
Copy Markdown
CollaboratorAuthor

Still needs a ticket.

This highlights that the builddir needs to be utilized differently. It needs to store all errors regardless if they are being suppressed. If cached results are found it will "replay" them and thus properly feed the suppressions (hopefully) allowing us to get rid of the hack introduced in #8173. This already feeds some cases where that hack was not sufficient.

This might not play well with CTU in some cases but if that is the case we are currently lacking the test coverage. I am approaching this in a test-driven way and this gets us a step closer to be able to merge #7079.

@firewavefirewave changed the title store all errors in AnalyzerInfo even if suppressedstore all errors in AnalyzerInformation even if suppressedFeb 26, 2026
@firewavefirewave changed the title store all errors in AnalyzerInformation even if suppressedfixed #14585 - store all errors in AnalyzerInformation even if suppressedMar 11, 2026
@firewave
firewave marked this pull request as ready for review March 11, 2026 14:59
@firewave

Copy link
Copy Markdown
CollaboratorAuthor

The added test might not be representative for all the issues it fixes. #7079 indicates there are multiple issues but I think it is fine to only have them implicitly tested (for now) when the other PR has finally been merged,

@sonarqubecloud

Copy link
Copy Markdown

@firewave
firewave merged commit 68c91ae into cppcheck-opensource:mainMar 15, 2026
76 of 84 checks passed
@firewave
firewave deleted the ainfo-suppr branch March 15, 2026 17:48
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

@firewave@danmar