diff options
author | Piotr Zegar <me@piotrzegar.pl> | 2024-03-26 18:20:25 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-03-26 18:20:25 +0100 |
commit | 313734795c67ca0bfb82f96b39e123774ddf24b4 (patch) | |
tree | cd38b8e3b3c4ed67d4096aeb1f556dacb7a5d33d /clang-tools-extra/test/clang-tidy/checkers/readability | |
parent | f4fc959c35eb862776ac2e83cb9f41aa88989e3f (diff) | |
download | llvm-313734795c67ca0bfb82f96b39e123774ddf24b4.zip llvm-313734795c67ca0bfb82f96b39e123774ddf24b4.tar.gz llvm-313734795c67ca0bfb82f96b39e123774ddf24b4.tar.bz2 |
[clang-tidy] Fix fix-it overlaps in readability-static-definition-in-anonymous-namespace (#86599)
Because check emitted multiple warnings for every template instance
fix-it couldn't be applied due to overlaps.
Using TK_IgnoreUnlessSpelledInSource and restricting check to C++ only.
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/checkers/readability')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/checkers/readability/static-definition-in-anonymous-namespace.cpp | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/clang-tools-extra/test/clang-tidy/checkers/readability/static-definition-in-anonymous-namespace.cpp b/clang-tools-extra/test/clang-tidy/checkers/readability/static-definition-in-anonymous-namespace.cpp index e9938db..e204199 100644 --- a/clang-tools-extra/test/clang-tidy/checkers/readability/static-definition-in-anonymous-namespace.cpp +++ b/clang-tools-extra/test/clang-tidy/checkers/readability/static-definition-in-anonymous-namespace.cpp @@ -51,6 +51,17 @@ static int c = 1; } // namespace deep_inner } // namespace inner +template<typename T> +static void printTemplate(T&&) {} +// CHECK-MESSAGES: :[[@LINE-1]]:13: warning: 'printTemplate' is a static definition in anonymous namespace; static is redundant here [readability-static-definition-in-anonymous-namespace] +// CHECK-FIXES: {{^}}void printTemplate(T&&) {} + +void testTemplate() { + printTemplate(5); + printTemplate(5U); + printTemplate("some string"); +} + } // namespace namespace N { |