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

added option for building with static analysis #616

Merged
merged 1 commit into from
Oct 24, 2023

Conversation

engelmi
Copy link
Member

@engelmi engelmi commented Oct 24, 2023

Related to: #604

Added the gcc option -fanalyzer as configurable meson option (default: false) to ensure higher code quality. Default is set to false and no integration into CI is done since it can also result in false-positives. Therefore, the developer has to decide which issue to tackle.

@engelmi engelmi requested review from dougsland and mwperina October 24, 2023 07:39
@engelmi engelmi force-pushed the add-gcc-option-fanalyzer branch 3 times, most recently from 33bd8b1 to 17b84c7 Compare October 24, 2023 07:41
Copy link
Member

@mwperina mwperina left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@engelmi engelmi force-pushed the add-gcc-option-fanalyzer branch from 17b84c7 to f4ba60b Compare October 24, 2023 11:34
Related to: #604

Added the gcc option -fanalyzer as configurable meson option
(default: false) to ensure higher code quality. Default is
set to false and no integration into CI is done since it can
also result in false-positives. Therefore, the developer has
to decide which issue to tackle.

Signed-off-by: Michael Engel <[email protected]>
@engelmi engelmi force-pushed the add-gcc-option-fanalyzer branch from f4ba60b to f2dc589 Compare October 24, 2023 12:01
@engelmi
Copy link
Member Author

engelmi commented Oct 24, 2023

Updated the table of contents in the README.developer.md as well.

@engelmi engelmi merged commit baafeaf into main Oct 24, 2023
@engelmi engelmi deleted the add-gcc-option-fanalyzer branch October 24, 2023 12:10
Sign up for free to 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