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

Make clang-tidy happy #20

Open
2 of 4 tasks
vasild opened this issue Jan 17, 2020 · 6 comments
Open
2 of 4 tasks

Make clang-tidy happy #20

vasild opened this issue Jan 17, 2020 · 6 comments

Comments

@vasild
Copy link
Contributor

vasild commented Jan 17, 2020

  • Get https://include-what-you-use.org/ happy with this project
    clang-tidy supersedes this
  • Integrate it either into make test or its own make iwyu
    clang-tidy is executed when compiling
  • Fix or suppress existent warnings; done in Address clang-tidy-19 warnings #144
  • Consider some CI to run it automatically per pull request to catch future regressions
@purpleKarrot
Copy link
Contributor

This may be regarded as obsolete/reduntant with clang-tidy's include-fixer check.

@vasild
Copy link
Contributor Author

vasild commented Feb 3, 2025

CMakeLists.txt only sets set(CMAKE_CXX_CLANG_TIDY "${CLANG_TIDY_EXECUTABLE}") and never actually executes clang-tidy?

@hebasto
Copy link
Contributor

hebasto commented Feb 3, 2025

CMakeLists.txt only sets set(CMAKE_CXX_CLANG_TIDY "${CLANG_TIDY_EXECUTABLE}") and never actually executes clang-tidy?

It does. You can check it out using --verbose.

@vasild
Copy link
Contributor Author

vasild commented Feb 3, 2025

Right!

src/mp/util.cpp:9:1: warning: included header array.h is not used directly [misc-include-cleaner]
    9 | #include <kj/array.h>
      | ^~~~~~~~~~~~~~~~~~~~~
   10 | #include <pthread.h>

I updated the OP.

@vasild vasild changed the title Integrate iwyu Make clang-tidy happy Feb 3, 2025
@hebasto
Copy link
Contributor

hebasto commented Feb 3, 2025

I updated the OP.

Did you see #144?

@vasild
Copy link
Contributor Author

vasild commented Feb 3, 2025

I updated the OP.

Did you see #144?

Before no, but now yes. Updated the OP even more :)

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

No branches or pull requests

3 participants