Skip to content

Updated CI to Clang 19#5945

Merged
chrchr-github merged 15 commits into
cppcheck-opensource:mainfrom
firewave:clang-19
Jul 30, 2024
Merged

Updated CI to Clang 19#5945
chrchr-github merged 15 commits into
cppcheck-opensource:mainfrom
firewave:clang-19

Conversation

@firewave

@firewave firewave commented Feb 4, 2024

Copy link
Copy Markdown
Collaborator

No description provided.

@firewave firewave force-pushed the clang-19 branch 2 times, most recently from 4ec0036 to 5573ae5 Compare February 12, 2024 00:56
@firewave firewave force-pushed the clang-19 branch 3 times, most recently from 8ed20dc to ecb4add Compare March 3, 2024 17:02
@firewave

firewave commented Mar 3, 2024

Copy link
Copy Markdown
Collaborator Author

I filed llvm/llvm-project#83732 about the modernize-use-designated-initializers false positives.

@firewave firewave force-pushed the clang-19 branch 2 times, most recently from 11485cc to afc4877 Compare March 4, 2024 00:23
@firewave firewave force-pushed the clang-19 branch 2 times, most recently from 1c1758c to 528a32b Compare April 7, 2024 17:32
@firewave firewave force-pushed the clang-19 branch 2 times, most recently from 9618787 to 3cb41b9 Compare June 12, 2024 08:48
@firewave

firewave commented Jul 11, 2024

Copy link
Copy Markdown
Collaborator Author

This needs tickets filed about the performance-unnecessary-copy-initialization case which are not detected by us.

@firewave

firewave commented Jul 11, 2024

Copy link
Copy Markdown
Collaborator Author

CheckOther::constVariableError() - https://trac.cppcheck.net/ticket/12925
ProjectFileDialog::loadFromProjectFile() - https://trac.cppcheck.net/ticket/12926 / https://trac.cppcheck.net/ticket/12934
The one in isTemporary() looks like a variation on this.

Comment thread gui/mainwindow.cpp Outdated
Comment thread lib/checkersreport.cpp Outdated
@firewave

Copy link
Copy Markdown
Collaborator Author

Clang 19 release candidate has been released and the apt mirror should also have made the branch switch. As we only utilize the binaries in the CI we can merge this now.

@firewave

Copy link
Copy Markdown
Collaborator Author

I filed llvm/llvm-project#101181 upstream about the performance-unnecessary-value-param false positive.

@firewave firewave marked this pull request as ready for review July 30, 2024 13:20
@chrchr-github chrchr-github merged commit 02e2230 into cppcheck-opensource:main Jul 30, 2024
@firewave firewave deleted the clang-19 branch July 30, 2024 19:35
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