fixed #14585 - store all errors in AnalyzerInformation even if suppressed#8267
fixed #14585 - store all errors in AnalyzerInformation even if suppressed#8267firewave wants to merge 1 commit intodanmar:mainfrom
AnalyzerInformation even if suppressed#8267Conversation
|
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. |
firewave
changed the title
AnalyzerInfo even if suppressedAnalyzerInformation even if suppressed
firewave
changed the title
AnalyzerInformation even if suppressedAnalyzerInformation even if suppressed
|
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, |
|
1 participant
No description provided.