diff --git a/.clang-tidy b/.clang-tidy index 0b130f31de08..1ae15b041d54 100644 --- a/.clang-tidy +++ b/.clang-tidy @@ -37,7 +37,6 @@ Checks: > -misc-non-private-member-variables-in-classes, -misc-throw-by-value-catch-by-reference, -misc-use-anonymous-namespace, - -modernize-avoid-c-arrays, -modernize-deprecated-ios-base-aliases, -misc-include-cleaner, -misc-unused-using-decls, @@ -77,3 +76,5 @@ CheckOptions: value: '1' - key: readability-simplify-boolean-expr.SimplifyDeMorgan value: '0' + - key: modernize-avoid-c-arrays.AllowStringArrays + value: true diff --git a/clang-tidy.md b/clang-tidy.md index 108af31a354e..dc9e4f4b322b 100644 --- a/clang-tidy.md +++ b/clang-tidy.md @@ -110,7 +110,7 @@ Produces warnings which might be considered false positives starting with C++11 `modernize-avoid-c-arrays`
-Produces warnings when `const char[]` is being used which is quite common in our code. Does not make sense to enable before C++17 when `std::string_view` becomes available. +Does not make sense to enable before C++17 when `std::string_view` becomes available. Also reports a false positive about templates which deduce the array length: https://github.com/llvm/llvm-project/issues/60053. `misc-include-cleaner`