Skip to content
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

enable --inline-suppr by default #79

Open
firewave opened this issue Aug 14, 2022 · 1 comment
Open

enable --inline-suppr by default #79

firewave opened this issue Aug 14, 2022 · 1 comment
Assignees

Comments

@firewave
Copy link
Collaborator

firewave commented Aug 14, 2022

Currently we do not enable the handling of inline suppressions by default. We should enable this since it just makes sense and also to avoid unexpected warnings from already annotated code.

There's a caveat though if the code is being processed by multiple Cppcheck versions and the finding is no longer reported. This might lead to unexpected unmatchedSuppression warnings (which was proposed in #77).

@firewave firewave self-assigned this Aug 15, 2022
@firewave
Copy link
Collaborator Author

firewave commented Jan 3, 2025

There's a caveat though if the code is being processed by multiple Cppcheck versions and the finding is no longer reported. This might lead to unexpected unmatchedSuppression warnings (which was proposed in #77).

Because of this it should only be implemented after unmatchedSuppression is no longer part of --enable=information which will hopefully be the default behavior starting with Cppcheck 2.18.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant