fixed #10697 - split unmatchedSuppression from --enable=information#7173
fixed #10697 - split unmatchedSuppression from --enable=information#7173firewave wants to merge 1 commit into
unmatchedSuppression from --enable=information#7173Conversation
|
First of all Secondly this is not very portable at all. Also depending on your configuration or just the underlying version of Cppcheck you will get these warnings. These might not apply to you so you want to get rid of them but they might show up for somebody else. So this is not something a regular user should not have enabled (this might also go for other messages in the category but one thing at a time). It is rather something you want to have within your CI. It seems like something that should go behind Also suppressing these is an annoying chicken and egg problem (at least used to be). In case you have a version which no longer or not yet triggers any of the findings which needs to be suppressed you will get these warnings and then you have to suppress them. But if you do that and the suppression is actually used then you get a message that there was no unmatched suppression. This is probably something that should be avoided but something to look into after the unmatched suppression are actually reported as intended. The reason the deprecation period is that short is because of the brokenness of this. |
a1dcb3f to
5b3d8ca
Compare
5b3d8ca to
3826bf8
Compare
|
|
As My second point of these being incorrectly reported because of the configuration used still is valid but we should wait a bit if we get reports about this. After all the improvements have only been available in recent releases and some distros/users might still need to catch up. It is also possible that people are never seeing these since |
|
It seems an actual use case has just come up where it would make sense to have it split out after all - see llvm/llvm-project#175417 (comment). |
|
Hi, I would also be interested in a better support for this check to be able to clean unnecessary suppression.
|



No description provided.