Skip to content

Refs #10958: add missing operators in valueFlowSameExpressions()#8350

Merged
chrchr-github merged 4 commits intocppcheck-opensource:mainfrom
chrchr-github:sameExpr
Mar 27, 2026
Merged

Refs #10958: add missing operators in valueFlowSameExpressions()#8350
chrchr-github merged 4 commits intocppcheck-opensource:mainfrom
chrchr-github:sameExpr

Conversation

@chrchr-github
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread lib/token.cpp
@sonarqubecloud
Copy link
Copy Markdown

@firewave
Copy link
Copy Markdown
Collaborator

Does this help with https://trac.cppcheck.net/ticket/13305?

@chrchr-github
Copy link
Copy Markdown
Collaborator Author

Does this help with https://trac.cppcheck.net/ticket/13305?

Unfortunately not, could be an easy fix though: if (tok->str() != "%") continue;

@chrchr-github chrchr-github marked this pull request as ready for review March 18, 2026 19:05
Copy link
Copy Markdown
Collaborator

@danmar danmar left a comment

Choose a reason for hiding this comment

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

we have some script in CI that runs the tests with match compiler also right?

@firewave
Copy link
Copy Markdown
Collaborator

we have some script in CI that runs the tests with match compiler also right?

Yes. Optimized builds in the CI use the matchcompiler (also for performance reasons).

@chrchr-github chrchr-github merged commit 67e9b55 into cppcheck-opensource:main Mar 27, 2026
72 checks passed
@chrchr-github chrchr-github deleted the sameExpr branch March 27, 2026 18:29
chrchr-github added a commit to chrchr-github/cppcheck that referenced this pull request Mar 31, 2026
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.

4 participants