martinus / nanobench

Simple, fast, accurate single-header microbenchmarking functionality for C++11/14/17/20
https://nanobench.ankerl.com
MIT License
1.43k stars 82 forks source link

.clang-tidy file is too open / not portable #108

Open maldag opened 6 months ago

maldag commented 6 months ago

The used .clang-tidy activates all checks and disables just some of it. Upon integrating nanobench in another project which also uses .clang-tidy, probably in a different version, creates problems. The other version may introduce new checks which are implicitly enabled and not taken care of.

My suggestion would be to either disable .clang-tidy for integrations in other projects or explicitly activate just the checks that are to be checked. Testet with clang-tidy 17

The following checks cause new problems:

-llvmlibc-inline-function-decl
-modernize-use-nodiscard,
-modernize-type-traits,
-readability-named-parameter
#and there are more ...