Simplify clang-tidy configuration

7 files changed, 60 insertions, 154 deletions

diff --git a/.clang-tidy b/.clang-tidy
index 241b17ed..69353fab 100644
--- a/.clang-tidy
+++ b/.clang-tidy
@@ -1,74 +1,32 @@
 Checks: >
   *,
-  -*-avoid-c-arrays,
   -*-magic-numbers,
   -*-named-parameter,
   -*-narrowing-conversions,
   -*-non-private-member-variables-in-classes,
-  -*-uppercase-literal-suffix,
-  -abseil-string-find-str-contains,
+  -*-special-member-functions,
+  -abseil-*,
   -altera-*,
-  -android-cloexec-*,
-  -bugprone-branch-clone,
-  -bugprone-easily-swappable-parameters,
-  -bugprone-exception-escape,
-  -bugprone-macro-parentheses,
-  -bugprone-parent-virtual-call,
-  -bugprone-reserved-identifier,
-  -bugprone-suspicious-string-compare,
-  -cert-dcl37-c,
-  -cert-dcl50-cpp,
-  -cert-dcl51-cpp,
-  -cert-err33-c,
-  -cert-err34-c,
-  -cert-err58-cpp,
-  -cert-str34-c,
-  -clang-analyzer-alpha.*,
-  -clang-analyzer-core.CallAndMessage,
-  -clang-analyzer-optin.cplusplus.VirtualCall,
-  -clang-analyzer-valist.Uninitialized,
-  -concurrency-mt-unsafe,
-  -cppcoreguidelines-avoid-non-const-global-variables,
   -cppcoreguidelines-macro-usage,
-  -cppcoreguidelines-no-malloc,
-  -cppcoreguidelines-owning-memory,
-  -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-reinterpret-cast,
-  -cppcoreguidelines-pro-type-static-cast-downcast,
-  -cppcoreguidelines-pro-type-union-access,
-  -cppcoreguidelines-pro-type-vararg,
-  -cppcoreguidelines-slicing,
-  -fuchsia-*,
+  -fuchsia-default-arguments-calls,
+  -fuchsia-default-arguments-declarations,
+  -fuchsia-multiple-inheritance,
+  -fuchsia-overloaded-operator,
   -google-default-arguments,
   -google-explicit-constructor,
-  -google-readability-casting,
-  -google-readability-todo,
-  -google-runtime-int,
-  -google-runtime-references,
   -hicpp-explicit-conversions,
-  -hicpp-multiway-paths-covered,
-  -hicpp-no-array-decay,
-  -hicpp-no-malloc,
   -hicpp-signed-bitwise,
-  -hicpp-vararg,
   -llvm-header-guard,
   -llvmlibc-*,
-  -misc-no-recursion,
   -misc-unused-parameters,
   -modernize-concat-nested-namespaces,
   -modernize-make-unique,
+  -modernize-pass-by-value,
   -modernize-return-braced-init-list,
   -modernize-use-nodiscard,
   -modernize-use-trailing-return-type,
-  -readability-convert-member-functions-to-static,
-  -readability-function-cognitive-complexity,
   -readability-identifier-length,
   -readability-implicit-bool-conversion,
-  -readability-use-anyofallof,
 CheckOptions:
   - key:   modernize-use-override.AllowOverrideAndFinal
     value: 'true'
diff --git a/src/.clang-tidy b/src/.clang-tidy
new file mode 100644
index 00000000..8b9aa445
--- /dev/null
+++ b/src/.clang-tidy
@@ -0,0 +1,29 @@
+Checks: >
+  -*-avoid-c-arrays,
+  -*-uppercase-literal-suffix,
+  -*-vararg,
+  -android-cloexec-*,
+  -bugprone-easily-swappable-parameters,
+  -cert-dcl50-cpp,
+  -cert-err33-c,
+  -cert-err34-c,
+  -clang-analyzer-optin.cplusplus.VirtualCall,
+  -clang-analyzer-valist.Uninitialized,
+  -concurrency-mt-unsafe,
+  -cppcoreguidelines-avoid-non-const-global-variables,
+  -cppcoreguidelines-no-malloc,
+  -cppcoreguidelines-owning-memory,
+  -cppcoreguidelines-pro-bounds-array-to-pointer-decay,
+  -cppcoreguidelines-pro-bounds-pointer-arithmetic,
+  -cppcoreguidelines-pro-type-const-cast,
+  -cppcoreguidelines-pro-type-reinterpret-cast,
+  -cppcoreguidelines-pro-type-union-access,
+  -fuchsia-statically-constructed-objects,
+  -google-readability-casting,
+  -hicpp-no-array-decay,
+  -hicpp-no-malloc,
+  -misc-no-recursion,
+  -misc-unused-parameters,
+  -readability-function-cognitive-complexity,
+  -readability-use-anyofallof,
+InheritParentConfig: true
diff --git a/src/client/.clang-tidy b/src/client/.clang-tidy
new file mode 100644
index 00000000..2561514f
--- /dev/null
+++ b/src/client/.clang-tidy
@@ -0,0 +1,3 @@
+Checks: >
+  -google-readability-todo,
+InheritParentConfig: true
diff --git a/src/gui/.clang-tidy b/src/gui/.clang-tidy
index 91428ef8..5f6e0c3f 100644
--- a/src/gui/.clang-tidy
+++ b/src/gui/.clang-tidy
@@ -1,16 +1,6 @@
 Checks: >
-  *,
-  -*-avoid-c-arrays,
-  -*-magic-numbers,
-  -*-named-parameter,
-  -*-narrowing-conversions,
-  -*-non-private-member-variables-in-classes,
-  -*-uppercase-literal-suffix,
-  -abseil-string-find-str-contains,
-  -altera-*,
   -android-cloexec-*,
   -bugprone-branch-clone,
-  -bugprone-easily-swappable-parameters,
   -bugprone-exception-escape,
   -bugprone-macro-parentheses,
   -bugprone-parent-virtual-call,
@@ -18,65 +8,22 @@ Checks: >
   -bugprone-suspicious-string-compare,
   -cert-dcl21-cpp,
   -cert-dcl37-c,
-  -cert-dcl50-cpp,
   -cert-dcl51-cpp,
-  -cert-err33-c,
-  -cert-err34-c,
   -cert-err58-cpp,
   -cert-str34-c,
-  -clang-analyzer-alpha.*,
   -clang-analyzer-core.CallAndMessage,
-  -clang-analyzer-optin.cplusplus.VirtualCall,
-  -clang-analyzer-valist.Uninitialized,
-  -concurrency-mt-unsafe,
-  -cppcoreguidelines-avoid-non-const-global-variables,
   -cppcoreguidelines-macro-usage,
-  -cppcoreguidelines-no-malloc,
-  -cppcoreguidelines-owning-memory,
-  -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-reinterpret-cast,
   -cppcoreguidelines-pro-type-static-cast-downcast,
-  -cppcoreguidelines-pro-type-union-access,
   -cppcoreguidelines-pro-type-vararg,
   -cppcoreguidelines-slicing,
-  -fuchsia-*,
   -google-default-arguments,
-  -google-explicit-constructor,
-  -google-readability-casting,
   -google-readability-todo,
   -google-runtime-int,
   -google-runtime-references,
-  -hicpp-explicit-conversions,
   -hicpp-multiway-paths-covered,
-  -hicpp-no-array-decay,
-  -hicpp-no-malloc,
-  -hicpp-signed-bitwise,
   -hicpp-vararg,
-  -llvm-header-guard,
-  -llvmlibc-*,
-  -misc-no-recursion,
-  -misc-unused-parameters,
   -modernize-avoid-bind,
-  -modernize-concat-nested-namespaces,
-  -modernize-make-unique,
-  -modernize-return-braced-init-list,
-  -modernize-use-nodiscard,
-  -modernize-use-trailing-return-type,
   -readability-convert-member-functions-to-static,
-  -readability-function-cognitive-complexity,
-  -readability-identifier-length,
-  -readability-implicit-bool-conversion,
-  -readability-use-anyofallof,
-CheckOptions:
-  - key:   modernize-use-override.AllowOverrideAndFinal
-    value: 'true'
-CheckOptions:
-  - key:   cppcoreguidelines-explicit-virtual-functions.AllowOverrideAndFinal
-    value: 'true'
-WarningsAsErrors: '*'
-HeaderFilterRegex: '.*'
-FormatStyle: file
+InheritParentConfig: true
diff --git a/src/ingen/.clang-tidy b/src/ingen/.clang-tidy
new file mode 100644
index 00000000..e7bf0b6a
--- /dev/null
+++ b/src/ingen/.clang-tidy
@@ -0,0 +1,3 @@
+Checks: >
+  -bugprone-exception-escape,
+InheritParentConfig: true
diff --git a/src/server/.clang-tidy b/src/server/.clang-tidy
index 8d9d65fa..51c9b102 100644
--- a/src/server/.clang-tidy
+++ b/src/server/.clang-tidy
@@ -1,72 +1,22 @@
 Checks: >
-  *,
   -*-avoid-c-arrays,
-  -*-magic-numbers,
-  -*-named-parameter,
-  -*-narrowing-conversions,
   -*-non-private-member-variables-in-classes,
   -*-uppercase-literal-suffix,
   -*-vararg,
-  -abseil-string-find-str-contains,
-  -altera-*,
-  -android-cloexec-*,
   -bugprone-branch-clone,
-  -bugprone-easily-swappable-parameters,
   -bugprone-parent-virtual-call,
   -bugprone-reserved-identifier,
   -bugprone-suspicious-string-compare,
   -cert-dcl37-c,
   -cert-dcl51-cpp,
-  -cert-err33-c,
-  -cert-str34-c,
-  -clang-analyzer-optin.cplusplus.VirtualCall,
-  -concurrency-mt-unsafe,
-  -cppcoreguidelines-avoid-non-const-global-variables,
-  -cppcoreguidelines-macro-usage,
-  -cppcoreguidelines-no-malloc,
-  -cppcoreguidelines-owning-memory,
-  -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-reinterpret-cast,
   -cppcoreguidelines-pro-type-static-cast-downcast,
-  -cppcoreguidelines-pro-type-union-access,
-  -fuchsia-*,
-  -google-default-arguments,
-  -google-explicit-constructor,
-  -google-readability-casting,
   -google-readability-todo,
   -google-runtime-int,
   -google-runtime-references,
-  -hicpp-explicit-conversions,
   -hicpp-multiway-paths-covered,
-  -hicpp-no-array-decay,
-  -hicpp-no-malloc,
-  -hicpp-signed-bitwise,
   -llvm-header-guard,
-  -llvmlibc-*,
-  -misc-no-recursion,
   -misc-redundant-expression,
-  -misc-unused-parameters,
-  -modernize-concat-nested-namespaces,
-  -modernize-make-unique,
-  -modernize-pass-by-value,
-  -modernize-return-braced-init-list,
-  -modernize-use-nodiscard,
-  -modernize-use-trailing-return-type,
   -portability-simd-intrinsics,
-  -readability-function-cognitive-complexity,
-  -readability-identifier-length,
-  -readability-implicit-bool-conversion,
-  -readability-use-anyofallof,
-CheckOptions:
-  - key:   modernize-use-override.AllowOverrideAndFinal
-    value: 'true'
-CheckOptions:
-  - key:   cppcoreguidelines-explicit-virtual-functions.AllowOverrideAndFinal
-    value: 'true'
-WarningsAsErrors: '*'
-HeaderFilterRegex: '.*'
-FormatStyle: file
+InheritParentConfig: true
diff --git a/tests/.clang-tidy b/tests/.clang-tidy
new file mode 100644
index 00000000..fb68f4e4
--- /dev/null
+++ b/tests/.clang-tidy
@@ -0,0 +1,16 @@
+Checks: >
+  -*-no-malloc,
+  -*-vararg,
+  -android-cloexec-fopen,
+  -cert-err33-c,
+  -concurrency-mt-unsafe,
+  -cppcoreguidelines-avoid-non-const-global-variables,
+  -cppcoreguidelines-owning-memory,
+  -cppcoreguidelines-pro-bounds-pointer-arithmetic,
+  -cppcoreguidelines-pro-type-reinterpret-cast,
+  -cppcoreguidelines-pro-type-union-access,
+  -fuchsia-statically-constructed-objects,
+  -google-readability-casting,
+  -google-readability-todo,
+  -readability-function-cognitive-complexity,
+InheritParentConfig: true