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

Fix warnings from cppcheck static analysis #8

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

escherstair
Copy link

No description provided.

@escherstair
Copy link
Author

@mgrella, @krt306, @tanujlokpal
can someone have a look to the proposal pull requests?
I can propose other improvements, but I'm not sure if someone on ST side is interested in them.

@tanujlokpal
Copy link
Collaborator

Hello @escherstair,

Thanks for your comment.

The checks and initializations in the code are necessary from functionality point of view and should not be changed.

Thanks

@escherstair
Copy link
Author

Hi @tanujlokpal
from the "workflow" perspective, nothing is changed in my PR.
They're only fixes suggested by cppcheck (i.e., removing "always true" condition, reduce scope of variable, ...).
They're important because my target is to have as much as possible "zero warnings" in my source code (and in the third party code I use).

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