Index: clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp +++ clang-tools-extra/clang-tidy/bugprone/BugproneTidyModule.cpp @@ -113,7 +113,8 @@ CheckFactories.registerCheck( "bugprone-multiple-statement-macro"); CheckFactories.registerCheck( - "bugprone-narrowing-conversions"); + "bugprone-narrowing-conversions", + "cppcoreguidelines-narrowing-conversions"); CheckFactories.registerCheck( "bugprone-not-null-terminated-result"); CheckFactories.registerCheck( Index: clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp +++ clang-tools-extra/clang-tidy/cert/CERTTidyModule.cpp @@ -45,19 +45,19 @@ "cert-dcl21-cpp"); CheckFactories.registerCheck("cert-dcl50-cpp"); CheckFactories.registerCheck( - "cert-dcl54-cpp"); + "cert-dcl54-cpp", "misc-new-delete-overloads"); CheckFactories.registerCheck( "cert-dcl58-cpp"); CheckFactories.registerCheck( - "cert-dcl59-cpp"); + "cert-dcl59-cpp", "google-build-namespaces"); // ERR CheckFactories.registerCheck( - "cert-err09-cpp"); + "cert-err09-cpp", "misc-throw-by-value-catch-by-reference"); CheckFactories.registerCheck("cert-err52-cpp"); CheckFactories.registerCheck("cert-err58-cpp"); CheckFactories.registerCheck("cert-err60-cpp"); CheckFactories.registerCheck( - "cert-err61-cpp"); + "cert-err61-cpp", "misc-throw-by-value-catch-by-reference"); // MEM CheckFactories.registerCheck( "cert-mem57-cpp"); @@ -67,23 +67,25 @@ "cert-msc51-cpp"); // OOP CheckFactories.registerCheck( - "cert-oop11-cpp"); + "cert-oop11-cpp", "performance-move-constructor-init"); CheckFactories.registerCheck( - "cert-oop54-cpp"); + "cert-oop54-cpp", "bugprone-unhandled-self-assignment"); CheckFactories.registerCheck( "cert-oop58-cpp"); // C checkers // DCL - CheckFactories.registerCheck("cert-dcl03-c"); + CheckFactories.registerCheck("cert-dcl03-c", + "misc-static-assert"); CheckFactories.registerCheck( - "cert-dcl16-c"); + "cert-dcl16-c", "readability-uppercase-literal-suffix"); // ENV CheckFactories.registerCheck("cert-env33-c"); // FLP CheckFactories.registerCheck("cert-flp30-c"); // FIO - CheckFactories.registerCheck("cert-fio38-c"); + CheckFactories.registerCheck( + "cert-fio38-c", "misc-non-copyable-objects"); // ERR CheckFactories.registerCheck("cert-err34-c"); // MSC @@ -92,7 +94,7 @@ "cert-msc32-c"); // POS CheckFactories.registerCheck( - "cert-pos44-c"); + "cert-pos44-c", "bugprone-bad-signal-to-kill-thread"); } ClangTidyOptions getModuleOptions() override { Index: clang-tools-extra/clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp +++ clang-tools-extra/clang-tidy/cppcoreguidelines/CppCoreGuidelinesTidyModule.cpp @@ -43,13 +43,14 @@ public: void addCheckFactories(ClangTidyCheckFactories &CheckFactories) override { CheckFactories.registerCheck( - "cppcoreguidelines-avoid-c-arrays"); + "cppcoreguidelines-avoid-c-arrays", "modernize-avoid-c-arrays"); CheckFactories.registerCheck( "cppcoreguidelines-avoid-goto"); CheckFactories.registerCheck( - "cppcoreguidelines-avoid-magic-numbers"); + "cppcoreguidelines-avoid-magic-numbers", "readability-magic-numbers"); CheckFactories.registerCheck( - "cppcoreguidelines-explicit-virtual-functions"); + "cppcoreguidelines-explicit-virtual-functions", + "modernize-use-override"); CheckFactories.registerCheck( "cppcoreguidelines-init-variables"); CheckFactories.registerCheck( @@ -60,7 +61,8 @@ "cppcoreguidelines-narrowing-conversions"); CheckFactories.registerCheck("cppcoreguidelines-no-malloc"); CheckFactories.registerCheck( - "cppcoreguidelines-non-private-member-variables-in-classes"); + "cppcoreguidelines-non-private-member-variables-in-classes", + "misc-non-private-member-variables-in-classes"); CheckFactories.registerCheck( "cppcoreguidelines-owning-memory"); CheckFactories.registerCheck( @@ -87,7 +89,8 @@ "cppcoreguidelines-special-member-functions"); CheckFactories.registerCheck("cppcoreguidelines-slicing"); CheckFactories.registerCheck( - "cppcoreguidelines-c-copy-assignment-signature"); + "cppcoreguidelines-c-copy-assignment-signature", + "misc-unconventional-assignment-operator"); } ClangTidyOptions getModuleOptions() override { Index: clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp +++ clang-tools-extra/clang-tidy/fuchsia/FuchsiaTidyModule.cpp @@ -34,7 +34,7 @@ CheckFactories.registerCheck( "fuchsia-default-arguments-declarations"); CheckFactories.registerCheck( - "fuchsia-header-anon-namespaces"); + "fuchsia-header-anon-namespaces", "google-build-namespaces"); CheckFactories.registerCheck( "fuchsia-multiple-inheritance"); CheckFactories.registerCheck( Index: clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp +++ clang-tools-extra/clang-tidy/google/GoogleTidyModule.cpp @@ -74,12 +74,14 @@ "google-readability-todo"); CheckFactories .registerCheck( - "google-readability-braces-around-statements"); + "google-readability-braces-around-statements", + "readability-braces-around-statements"); CheckFactories.registerCheck( - "google-readability-function-size"); + "google-readability-function-size", "readability-function-size"); CheckFactories .registerCheck( - "google-readability-namespace-comments"); + "google-readability-namespace-comments", + "readability-namespace-comments"); CheckFactories.registerCheck( "google-upgrade-googletest-case"); } Index: clang-tools-extra/clang-tidy/hicpp/HICPPTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/hicpp/HICPPTidyModule.cpp +++ clang-tools-extra/clang-tidy/hicpp/HICPPTidyModule.cpp @@ -49,64 +49,67 @@ public: void addCheckFactories(ClangTidyCheckFactories &CheckFactories) override { CheckFactories.registerCheck( - "hicpp-avoid-c-arrays"); + "hicpp-avoid-c-arrays", "modernize-avoid-c-arrays"); CheckFactories.registerCheck( - "hicpp-avoid-goto"); + "hicpp-avoid-goto", "cppcoreguidelines-avoid-goto"); CheckFactories.registerCheck( - "hicpp-braces-around-statements"); + "hicpp-braces-around-statements", + "readability-braces-around-statements"); CheckFactories.registerCheck( - "hicpp-deprecated-headers"); + "hicpp-deprecated-headers", "modernize-deprecated-headers"); CheckFactories.registerCheck( "hicpp-exception-baseclass"); CheckFactories.registerCheck( "hicpp-multiway-paths-covered"); CheckFactories.registerCheck("hicpp-signed-bitwise"); CheckFactories.registerCheck( - "hicpp-explicit-conversions"); + "hicpp-explicit-conversions", "google-explicit-constructor"); CheckFactories.registerCheck( - "hicpp-function-size"); + "hicpp-function-size", "readability-function-size"); CheckFactories.registerCheck( - "hicpp-named-parameter"); + "hicpp-named-parameter", "readability-identifier-naming"); CheckFactories.registerCheck( - "hicpp-invalid-access-moved"); + "hicpp-invalid-access-moved", "bugprone-use-after-move"); CheckFactories.registerCheck( - "hicpp-member-init"); + "hicpp-member-init", "cppcoreguidelines-pro-type-member-init"); CheckFactories.registerCheck( - "hicpp-move-const-arg"); + "hicpp-move-const-arg", "performance-move-const-arg"); CheckFactories.registerCheck( - "hicpp-new-delete-operators"); + "hicpp-new-delete-operators", "misc-new-delete-operators"); CheckFactories.registerCheck( - "hicpp-noexcept-move"); + "hicpp-noexcept-move", "performance-noexcept-move-constructor"); CheckFactories .registerCheck( "hicpp-no-array-decay"); CheckFactories.registerCheck("hicpp-no-assembler"); CheckFactories.registerCheck( - "hicpp-no-malloc"); + "hicpp-no-malloc", "cppcoreguidelines-no-malloc"); CheckFactories .registerCheck( "hicpp-special-member-functions"); - CheckFactories.registerCheck( - "hicpp-static-assert"); - CheckFactories.registerCheck("hicpp-use-auto"); + CheckFactories.registerCheck("hicpp-static-assert", + "misc-static-assert"); + CheckFactories.registerCheck("hicpp-use-auto", + "modernize-use-auto"); CheckFactories.registerCheck( - "hicpp-undelegated-constructor"); + "hicpp-undelegated-constructor", "bugprone-undelegated-constructor"); CheckFactories.registerCheck( - "hicpp-use-emplace"); + "hicpp-use-emplace", "modernize-use-emplace"); CheckFactories.registerCheck( - "hicpp-use-equals-default"); + "hicpp-use-equals-default", "modernize-use-equals-default"); CheckFactories.registerCheck( - "hicpp-use-equals-delete"); + "hicpp-use-equals-delete", "modernize-use-equals-delete"); CheckFactories.registerCheck( - "hicpp-use-noexcept"); + "hicpp-use-noexcept", "modernize-use-noexcept"); CheckFactories.registerCheck( - "hicpp-use-nullptr"); + "hicpp-use-nullptr", "modernize-use-nullptr"); CheckFactories.registerCheck( - "hicpp-use-override"); + "hicpp-use-override", "modernize-use-override"); CheckFactories.registerCheck( - "hicpp-uppercase-literal-suffix"); + "hicpp-uppercase-literal-suffix", + "readability-uppercase-literal-suffix"); CheckFactories.registerCheck( - "hicpp-vararg"); + "hicpp-vararg", "cppcoreguidelines-pro-type-vararg"); } }; Index: clang-tools-extra/clang-tidy/llvm/LLVMTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/llvm/LLVMTidyModule.cpp +++ clang-tools-extra/clang-tidy/llvm/LLVMTidyModule.cpp @@ -26,7 +26,7 @@ CheckFactories.registerCheck("llvm-header-guard"); CheckFactories.registerCheck("llvm-include-order"); CheckFactories.registerCheck( - "llvm-namespace-comment"); + "llvm-namespace-comment", "readability-namespace-comment"); CheckFactories.registerCheck( "llvm-prefer-isa-or-dyn-cast-in-conditionals"); CheckFactories.registerCheck( Index: clang-tools-extra/clang-tidy/readability/ReadabilityTidyModule.cpp =================================================================== --- clang-tools-extra/clang-tidy/readability/ReadabilityTidyModule.cpp +++ clang-tools-extra/clang-tidy/readability/ReadabilityTidyModule.cpp @@ -102,7 +102,7 @@ "readability-static-definition-in-anonymous-namespace"); CheckFactories.registerCheck( "readability-string-compare"); - CheckFactories.registerCheck( + CheckFactories.registerCheck( "readability-named-parameter"); CheckFactories.registerCheck( "readability-non-const-parameter"); Index: clang-tools-extra/test/clang-tidy/infrastructure/duplicate-reports.cpp =================================================================== --- clang-tools-extra/test/clang-tidy/infrastructure/duplicate-reports.cpp +++ clang-tools-extra/test/clang-tidy/infrastructure/duplicate-reports.cpp @@ -1,15 +1,30 @@ -// RUN: %check_clang_tidy %s cert-err09-cpp,cert-err61-cpp %t -- -- -fexceptions +// RUN: %check_clang_tidy %s "cert-err09-cpp,cert-err61-cpp, \ +// RUN: google-readability-braces-around-statements, \ +// RUN: readability-braces-around-statements, \ +// RUN: hicpp=braces-around-statements" %t -- -- -fexceptions void alwaysThrows() { int ex = 42; - // CHECK-MESSAGES: warning: throw expression should throw anonymous temporary values instead [cert-err09-cpp] - // CHECK-MESSAGES: warning: throw expression should throw anonymous temporary values instead [cert-err61-cpp] throw ex; + // CHECK-MESSAGES: :[[@LINE-1]]:9: warning: throw expression should throw anonymous temporary values instead + // ^^ Dont specify which check as could be either. + // Original name is misc-throw-by-value-catch-by-reference. } void doTheJob() { try { alwaysThrows(); - } catch (int&) { + } catch (int &) { } } + +void bracesStatements() { + for (;;) + ; + // CHECK-MESSAGES: :[[@LINE-2]]:11: warning: statement should be inside braces [readability-braces-around-statements] + // Specify check name as readability-braces-around-statements is the non + // aliased name of the check + // CHECK-FIXES: {{^}} for (;;) { + // CHECK-FIXES-NEXT: {{^}} ; + // CHECK-FIXES-NEXT: {{^}}} +}