thevar1able/ClickHouse
1
0
Fork 0
mirror of https://github.com/ClickHouse/ClickHouse.git synced 2024-11-14 11:33:46 +00:00
Code Issues Packages Projects Releases Wiki Activity
350ff96628
ClickHouse/.clang-tidy

195 lines
6.2 KiB
Plaintext
Raw Normal View History

Enable clang-tidy for headers clang-tidy now also checks code in header files. Because the analyzer finds tons of issues, activate the check only for directory "base/" (see file ".clang-tidy"). All other directories, in particular "src/" are left to future work. While many findings were fixed, some were not (and suppressed instead). Reasons for this include: a) the file is 1:1 copypaste of a 3rd-party lib (e.g. pcg_extras.h) and fixing stuff would make upgrades/fixes more difficult b) a fix would have broken lots of using code
2022-08-26 13:54:57 +00:00
# To run clang-tidy from CMake, build ClickHouse with -DENABLE_CLANG_TIDY=1. To show all warnings, it is
# recommended to pass "-k0" to Ninja.
Add a comment about disabling further checks
2022-05-19 15:39:53 +00:00
# Enable all checks + disale selected checks. Feel free to remove disabled checks from below list if
# a) the new check is not controversial (this includes many checks in readability-* and google-*) or
# b) too noisy (checks with > 100 new warnings are considered noisy, this includes e.g. cppcoreguidelines-*).
Enable clang-tidy for headers clang-tidy now also checks code in header files. Because the analyzer finds tons of issues, activate the check only for directory "base/" (see file ".clang-tidy"). All other directories, in particular "src/" are left to future work. While many findings were fixed, some were not (and suppressed instead). Reasons for this include: a) the file is 1:1 copypaste of a 3rd-party lib (e.g. pcg_extras.h) and fixing stuff would make upgrades/fixes more difficult b) a fix would have broken lots of using code
2022-08-26 13:54:57 +00:00
# TODO Let clang-tidy check headers in further directories
# --> HeaderFilterRegex: '^.*/(src|base|programs|utils)/.*(h|hpp)$'
HeaderFilterRegex: '^.*/(base)/.*(h|hpp)$'
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
Checks: '*,
-abseil-*,
-altera-*,
-android-*,
-bugprone-branch-clone,
-bugprone-easily-swappable-parameters,
-bugprone-exception-escape,
-bugprone-implicit-widening-of-multiplication-result,
-bugprone-narrowing-conversions,
-bugprone-not-null-terminated-result,
Fix clang-tidy warnings (from clang-15) Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-09-10 07:58:49 +00:00
-bugprone-unchecked-optional-access,
-bugprone-assignment-in-if-condition,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
Fixup clang-tidy
2022-05-05 07:50:57 +00:00
-cert-dcl16-c,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-cert-err58-cpp,
Fixup clang-tidy
2022-05-05 07:50:57 +00:00
-cert-msc32-c,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-cert-msc51-cpp,
-cert-oop54-cpp,
-cert-oop57-cpp,
Further fixup of Linux-specific failures
2022-05-05 10:44:08 +00:00
-clang-analyzer-optin.performance.Padding,
Fixup clang-tidy
2022-05-05 07:50:57 +00:00
-clang-analyzer-optin.portability.UnixAPI,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-clang-analyzer-security.insecureAPI.bzero,
-clang-analyzer-security.insecureAPI.strcpy,
A few clangtidy updates Enable: - bugprone-lambda-function-name: "Checks for attempts to get the name of a function from within a lambda expression. The name of a lambda is always something like operator(), which is almost never what was intended." - bugprone-unhandled-self-assignment: "Finds user-defined copy assignment operators which do not protect the code against self-assignment either by checking self-assignment explicitly or using the copy-and-swap or the copy-and-move method."" - hicpp-invalid-access-moved: "Warns if an object is used after it has been moved." - hicpp-use-noexcept: "This check replaces deprecated dynamic exception specifications with the appropriate noexcept specification (introduced in C++11)" - hicpp-use-override: "Adds override (introduced in C++11) to overridden virtual functions and removes virtual from those functions as it is not required." - performance-type-promotion-in-math-fn: "Finds calls to C math library functions (from math.h or, in C++, cmath) with implicit float to double promotions." Split up: - cppcoreguidelines-*. Some of them may be useful (haven't checked in detail), therefore allow to toggle them individually. Disable: - linuxkernel-*. Obvious.
2022-05-16 15:44:19 +00:00
-cppcoreguidelines-avoid-c-arrays,
-cppcoreguidelines-avoid-goto,
-cppcoreguidelines-avoid-magic-numbers,
-cppcoreguidelines-avoid-non-const-global-variables,
-cppcoreguidelines-explicit-virtual-functions,
-cppcoreguidelines-init-variables,
-cppcoreguidelines-interfaces-global-init,
-cppcoreguidelines-macro-usage,
-cppcoreguidelines-narrowing-conversions,
-cppcoreguidelines-no-malloc,
-cppcoreguidelines-non-private-member-variables-in-classes,
-cppcoreguidelines-owning-memory,
-cppcoreguidelines-prefer-member-initializer,
-cppcoreguidelines-pro-bounds-array-to-pointer-decay,
-cppcoreguidelines-pro-bounds-constant-array-index,
-cppcoreguidelines-pro-bounds-pointer-arithmetic,
-cppcoreguidelines-pro-type-const-cast,
-cppcoreguidelines-pro-type-cstyle-cast,
-cppcoreguidelines-pro-type-member-init,
-cppcoreguidelines-pro-type-reinterpret-cast,
-cppcoreguidelines-pro-type-static-cast-downcast,
-cppcoreguidelines-pro-type-union-access,
-cppcoreguidelines-pro-type-vararg,
-cppcoreguidelines-slicing,
-cppcoreguidelines-special-member-functions,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-darwin-*,
-fuchsia-*,
-google-build-using-namespace,
-google-readability-braces-around-statements,
Merge branch 'master' into llvm-14
2022-05-16 10:03:26 +00:00
-google-readability-casting,
Fixup clang-tidy
2022-05-05 07:50:57 +00:00
-google-readability-function-size,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-google-readability-namespace-comments,
-google-readability-todo,
-google-upgrade-googletest-case,
-hicpp-avoid-c-arrays,
-hicpp-avoid-goto,
-hicpp-braces-around-statements,
Fixup clang-tidy
2022-05-05 07:50:57 +00:00
-hicpp-explicit-conversions,
-hicpp-function-size,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-hicpp-member-init,
-hicpp-move-const-arg,
-hicpp-multiway-paths-covered,
-hicpp-named-parameter,
-hicpp-no-array-decay,
-hicpp-no-assembler,
-hicpp-no-malloc,
-hicpp-signed-bitwise,
-hicpp-special-member-functions,
-hicpp-uppercase-literal-suffix,
-hicpp-use-auto,
-hicpp-use-emplace,
-hicpp-vararg,
A few clangtidy updates Enable: - bugprone-lambda-function-name: "Checks for attempts to get the name of a function from within a lambda expression. The name of a lambda is always something like operator(), which is almost never what was intended." - bugprone-unhandled-self-assignment: "Finds user-defined copy assignment operators which do not protect the code against self-assignment either by checking self-assignment explicitly or using the copy-and-swap or the copy-and-move method."" - hicpp-invalid-access-moved: "Warns if an object is used after it has been moved." - hicpp-use-noexcept: "This check replaces deprecated dynamic exception specifications with the appropriate noexcept specification (introduced in C++11)" - hicpp-use-override: "Adds override (introduced in C++11) to overridden virtual functions and removes virtual from those functions as it is not required." - performance-type-promotion-in-math-fn: "Finds calls to C math library functions (from math.h or, in C++, cmath) with implicit float to double promotions." Split up: - cppcoreguidelines-*. Some of them may be useful (haven't checked in detail), therefore allow to toggle them individually. Disable: - linuxkernel-*. Obvious.
2022-05-16 15:44:19 +00:00
-linuxkernel-*,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-llvm-*,
-llvmlibc-*,
-openmp-*,
-misc-no-recursion,
-misc-non-private-member-variables-in-classes,
Fix clang-tidy warnings (from clang-15) Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-09-10 07:58:49 +00:00
-misc-const-correctness,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-modernize-avoid-c-arrays,
-modernize-concat-nested-namespaces,
-modernize-pass-by-value,
-modernize-return-braced-init-list,
-modernize-use-auto,
-modernize-use-default-member-init,
-modernize-use-emplace,
-modernize-use-nodiscard,
-modernize-use-override,
-modernize-use-trailing-return-type,
Fix clang-tidy warnings (from clang-15) Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-09-10 07:58:49 +00:00
-modernize-macro-to-enum,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-performance-inefficient-string-concatenation,
-performance-no-int-to-ptr,
-performance-unnecessary-value-param,
Further fixup of Linux-specific failures
2022-05-05 10:44:08 +00:00
-portability-simd-intrinsics,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-readability-braces-around-statements,
-readability-else-after-return,
-readability-function-cognitive-complexity,
-readability-function-size,
Update .clang-tidy
2022-05-16 14:20:23 +00:00
-readability-identifier-length,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-readability-implicit-bool-conversion,
-readability-isolate-declaration,
-readability-magic-numbers,
-readability-named-parameter,
-readability-redundant-declaration,
-readability-static-accessed-through-instance,
-readability-suspicious-call-argument,
-readability-uppercase-literal-suffix,
-readability-use-anyofallof,
Fix clang-tidy warnings (from clang-15) Signed-off-by: Azat Khuzhin <a.khuzhin@semrush.com>
2022-09-10 07:58:49 +00:00
-readability-simplify-boolean-expr,
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
-zirkon-*,
clang-tidy, part 2
2020-03-18 00:57:00 +00:00
'
Invert .clang-tidy Instead of disabling all checks + specifying which are enabled (whitelist), enable all checks + specify which checks are disabled (blacklist). The same set of checks executes as before. Motivation: a) Given the goal to (generally) enable as many checks as possible, a blacklist produces a smaller and easier to maintain .clang-tidy file. b) Many new checks introduced with LLVM 12/13/14 were forgotten to turn on. A blacklist makes sure that future compiler upgrades enable new checks by default.
2022-05-04 17:24:02 +00:00
Added some clang-tidy checks
2020-03-08 20:17:49 +00:00
WarningsAsErrors: '*'
Clang Tidy, part 7 (#9799) * Attempt to enable identifier naming check * Attempt to enable identifier naming check, continue * Progress on identifier names * Adopt identifier names check * Fixed error * Merge with master * Fixed build * Fixed build * Fixed build * Fixed build * Fixed build * Fixed error * Fixed error * Fixed error
2020-03-23 02:12:31 +00:00
CheckOptions:
- key: readability-identifier-naming.ClassCase
value: CamelCase
- key: readability-identifier-naming.EnumCase
value: CamelCase
- key: readability-identifier-naming.LocalVariableCase
value: lower_case
- key: readability-identifier-naming.StaticConstantCase
value: aNy_CasE
- key: readability-identifier-naming.MemberCase
value: lower_case
- key: readability-identifier-naming.PrivateMemberPrefix
value: ''
- key: readability-identifier-naming.ProtectedMemberPrefix
value: ''
- key: readability-identifier-naming.PublicMemberCase
value: lower_case
- key: readability-identifier-naming.MethodCase
value: camelBack
- key: readability-identifier-naming.PrivateMethodPrefix
value: ''
- key: readability-identifier-naming.ProtectedMethodPrefix
value: ''
- key: readability-identifier-naming.ParameterPackCase
value: lower_case
- key: readability-identifier-naming.StructCase
value: CamelCase
- key: readability-identifier-naming.TemplateTemplateParameterCase
value: CamelCase
- key: readability-identifier-naming.TemplateUsingCase
value: lower_case
- key: readability-identifier-naming.TypeTemplateParameterCase
value: CamelCase
- key: readability-identifier-naming.TypedefCase
value: CamelCase
- key: readability-identifier-naming.UnionCase
value: CamelCase
- key: readability-identifier-naming.UsingCase
value: CamelCase
Revert "Revert "Fix tidy"" This reverts commit 2e5e017d6d9adbab69de388eb5cad88bfcf4310b.
2021-09-06 09:16:52 +00:00
- key: modernize-loop-convert.UseCxx20ReverseRanges
value: false
Revert "Fix errors of CheckTriviallyCopyableMove type"
2022-06-07 11:53:10 +00:00
- key: performance-move-const-arg.CheckTriviallyCopyableMove
value: false
Attempt to workaround clang-tidy bug
2022-03-10 17:30:46 +00:00
# Workaround clang-tidy bug: https://github.com/llvm/llvm-project/issues/46097
- key: readability-identifier-naming.TypeTemplateParameterIgnoredRegexp
value: expr-type
Reference in New Issue Copy Permalink