Skip to content

run clang-tidy with C++20#8188

Open
firewave wants to merge 3 commits intodanmar:mainfrom
firewave:tidy-cxx17
Open

run clang-tidy with C++20#8188
firewave wants to merge 3 commits intodanmar:mainfrom
firewave:tidy-cxx17

Conversation

@firewave
Copy link
Collaborator

@firewave firewave commented Feb 8, 2026

No description provided.

@firewave firewave changed the title run clang-tidy with C++17 run clang-tidy with C++20 Feb 8, 2026
@firewave
Copy link
Collaborator Author

firewave commented Feb 8, 2026

The idea here was that we might be able to address some of the findings by using conditional code.

@firewave firewave marked this pull request as ready for review February 26, 2026 08:46
@sonarqubecloud
Copy link

Copy link
Owner

@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.

I created this forum thread:
https://sourceforge.net/p/cppcheck/discussion/development/thread/34ff9ac443/

Please turn off the auto-warnings from clang-tidy it's too pedantic. and then we can use auto where we all agree that it is wanted.

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