Skip to content

Fix #14533, #14536 FN stlcstrConcat, stlcstrAssignment, stlcstrConstructor#8261

Merged
chrchr-github merged 16 commits intocppcheck-opensource:mainfrom
chrchr-github:chr_14533
Mar 11, 2026
Merged

Fix #14533, #14536 FN stlcstrConcat, stlcstrAssignment, stlcstrConstructor#8261
chrchr-github merged 16 commits intocppcheck-opensource:mainfrom
chrchr-github:chr_14533

Conversation

@chrchr-github
Copy link
Copy Markdown
Collaborator

No description provided.

Comment thread lib/checkstl.cpp Outdated
@chrchr-github chrchr-github changed the title Fix #14533 FN stlcstrConcat with constructor call Fix #14533, #14536 FN stlcstrConcat Feb 25, 2026
Comment thread lib/checkstl.cpp Outdated
@chrchr-github chrchr-github marked this pull request as draft February 26, 2026 08:47
@chrchr-github chrchr-github changed the title Fix #14533, #14536 FN stlcstrConcat Fix #14533, #14536 FN stlcstrConcat, stlcstrAssignment, stlcstrConstructor Feb 26, 2026
Comment thread gui/librarydialog.cpp
@chrchr-github chrchr-github marked this pull request as ready for review February 26, 2026 12:19
Comment thread lib/checkstl.cpp Outdated
Comment thread test/teststl.cpp
Comment thread lib/checkstl.cpp Outdated
@firewave

This comment was marked as resolved.

@sonarqubecloud
Copy link
Copy Markdown

Comment thread lib/checkstl.cpp Outdated
{
if (!tok->isBinaryOp() || !Token::simpleMatch(tok, "+"))
return false;
for (const Token* op : { tok->astOperand1(), tok->astOperand2() }) {
Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

Upstream issue: llvm/llvm-project#185593

@chrchr-github chrchr-github merged commit 1b9124e into cppcheck-opensource:main Mar 11, 2026
70 checks passed
@chrchr-github chrchr-github deleted the chr_14533 branch March 11, 2026 17:42
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