diff options
author | Alexander Kornienko <alexfh@google.com> | 2014-10-26 01:41:14 +0000 |
---|---|---|
committer | Alexander Kornienko <alexfh@google.com> | 2014-10-26 01:41:14 +0000 |
commit | 2192a8e5193716613bb9d64090a8843f7e5ca7fb (patch) | |
tree | 9d697a594076c9629ae0f9b89a4703317aab77b0 /clang-tools-extra/test | |
parent | f0ba6ce309d9ab412982e71cda9b3981e25625fd (diff) | |
download | bcm5719-llvm-2192a8e5193716613bb9d64090a8843f7e5ca7fb.tar.gz bcm5719-llvm-2192a8e5193716613bb9d64090a8843f7e5ca7fb.zip |
[clang-tidy] Bring order to check registration.
Summary:
Register readability checks in a separate module. Renamed the checks
and test file names accordingly.
Reviewers: djasper, klimek
Reviewed By: klimek
Subscribers: curdeius, cfe-commits
Differential Revision: http://reviews.llvm.org/D5936
llvm-svn: 220631
Diffstat (limited to 'clang-tools-extra/test')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/google-module.cpp | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/google-readability-casting.c (renamed from clang-tools-extra/test/clang-tidy/avoid-c-style-casts.c) | 0 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/google-readability-casting.cpp (renamed from clang-tools-extra/test/clang-tidy/avoid-c-style-casts.cpp) | 0 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/misc-use-override.cpp (renamed from clang-tools-extra/test/clang-tidy/use-override.cpp) | 0 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-braces-around-statements-few-lines.cpp (renamed from clang-tools-extra/test/clang-tidy/misc-braces-around-statements-few-lines.cpp) | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-braces-around-statements-same-line.cpp (renamed from clang-tools-extra/test/clang-tidy/misc-braces-around-statements-same-line.cpp) | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-braces-around-statements-single-line.cpp (renamed from clang-tools-extra/test/clang-tidy/misc-braces-around-statements-single-line.cpp) | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-braces-around-statements.cpp (renamed from clang-tools-extra/test/clang-tidy/misc-braces-around-statements.cpp) | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-function-size.cpp (renamed from clang-tools-extra/test/clang-tidy/misc-function-size.cpp) | 2 | ||||
-rw-r--r-- | clang-tools-extra/test/clang-tidy/readability-redundant-smartptr-get.cpp (renamed from clang-tools-extra/test/clang-tidy/redundant-smartptr-get.cpp) | 4 |
10 files changed, 9 insertions, 7 deletions
diff --git a/clang-tools-extra/test/clang-tidy/google-module.cpp b/clang-tools-extra/test/clang-tidy/google-module.cpp index b7be164a95e..b8c5d6949d2 100644 --- a/clang-tools-extra/test/clang-tidy/google-module.cpp +++ b/clang-tools-extra/test/clang-tidy/google-module.cpp @@ -2,6 +2,8 @@ // CHECK: CheckOptions: // CHECK: {{- key: *google-readability-braces-around-statements.ShortStatementLines}} // CHECK-NEXT: {{value: *'1'}} +// CHECK: {{- key: *google-readability-function-size.StatementThreshold}} +// CHECK-NEXT: {{value: *'800'}} // CHECK: {{- key: *google-readability-namespace-comments.ShortNamespaceLines}} // CHECK-NEXT: {{value: *'1'}} // CHECK: {{- key: *google-readability-namespace-comments.SpacesBeforeComments}} diff --git a/clang-tools-extra/test/clang-tidy/avoid-c-style-casts.c b/clang-tools-extra/test/clang-tidy/google-readability-casting.c index 87d1dfcdf42..87d1dfcdf42 100644 --- a/clang-tools-extra/test/clang-tidy/avoid-c-style-casts.c +++ b/clang-tools-extra/test/clang-tidy/google-readability-casting.c diff --git a/clang-tools-extra/test/clang-tidy/avoid-c-style-casts.cpp b/clang-tools-extra/test/clang-tidy/google-readability-casting.cpp index 414fa8bbcab..414fa8bbcab 100644 --- a/clang-tools-extra/test/clang-tidy/avoid-c-style-casts.cpp +++ b/clang-tools-extra/test/clang-tidy/google-readability-casting.cpp diff --git a/clang-tools-extra/test/clang-tidy/use-override.cpp b/clang-tools-extra/test/clang-tidy/misc-use-override.cpp index 63739ad4223..63739ad4223 100644 --- a/clang-tools-extra/test/clang-tidy/use-override.cpp +++ b/clang-tools-extra/test/clang-tidy/misc-use-override.cpp diff --git a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-few-lines.cpp b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-few-lines.cpp index 312b762ec45..d4a878104e9 100644 --- a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-few-lines.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-few-lines.cpp @@ -1,4 +1,4 @@ -// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s misc-braces-around-statements %t -config="{CheckOptions: [{key: misc-braces-around-statements.ShortStatementLines, value: 4}]}" -- +// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s readability-braces-around-statements %t -config="{CheckOptions: [{key: readability-braces-around-statements.ShortStatementLines, value: 4}]}" -- // REQUIRES: shell void do_something(const char *) {} diff --git a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-same-line.cpp b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-same-line.cpp index 4f1511258f3..a4a32ca44db 100644 --- a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-same-line.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-same-line.cpp @@ -1,4 +1,4 @@ -// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s misc-braces-around-statements %t -config="{CheckOptions: [{key: misc-braces-around-statements.ShortStatementLines, value: 1}]}" -- +// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s readability-braces-around-statements %t -config="{CheckOptions: [{key: readability-braces-around-statements.ShortStatementLines, value: 1}]}" -- // REQUIRES: shell void do_something(const char *) {} diff --git a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-single-line.cpp b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-single-line.cpp index 23b435f4c33..a9bf1416c04 100644 --- a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements-single-line.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements-single-line.cpp @@ -1,4 +1,4 @@ -// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s misc-braces-around-statements %t -config="{CheckOptions: [{key: misc-braces-around-statements.ShortStatementLines, value: 2}]}" -- +// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s readability-braces-around-statements %t -config="{CheckOptions: [{key: readability-braces-around-statements.ShortStatementLines, value: 2}]}" -- // REQUIRES: shell void do_something(const char *) {} diff --git a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements.cpp b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements.cpp index bdfd6aa1190..97bd05b9a28 100644 --- a/clang-tools-extra/test/clang-tidy/misc-braces-around-statements.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-braces-around-statements.cpp @@ -1,4 +1,4 @@ -// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s misc-braces-around-statements %t +// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s readability-braces-around-statements %t // REQUIRES: shell void do_something(const char *) {} diff --git a/clang-tools-extra/test/clang-tidy/misc-function-size.cpp b/clang-tools-extra/test/clang-tidy/readability-function-size.cpp index e0b24215723..526e272de7f 100644 --- a/clang-tools-extra/test/clang-tidy/misc-function-size.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-function-size.cpp @@ -1,7 +1,7 @@ // RUN: rm -rf %t // RUN: mkdir -p %t // RUN: sed 's#// *[A-Z-][A-Z-]*:.*#//#' %s > %t/t.cpp -// RUN: echo '{ Checks: "-*,misc-function-size", CheckOptions: [{key: misc-function-size.LineThreshold, value: 0}, {key: misc-function-size.StatementThreshold, value: 0}, {key: misc-function-size.BranchThreshold, value: 0}]}' > %t/.clang-tidy +// RUN: echo '{ Checks: "-*,readability-function-size", CheckOptions: [{key: readability-function-size.LineThreshold, value: 0}, {key: readability-function-size.StatementThreshold, value: 0}, {key: readability-function-size.BranchThreshold, value: 0}]}' > %t/.clang-tidy // RUN: clang-tidy %t/t.cpp -- -std=c++11 2>&1 | FileCheck %s -implicit-check-not='{{warning:|error:|note:}}' void foo1() { diff --git a/clang-tools-extra/test/clang-tidy/redundant-smartptr-get.cpp b/clang-tools-extra/test/clang-tidy/readability-redundant-smartptr-get.cpp index 7f698bf6547..c69e1fa39c5 100644 --- a/clang-tools-extra/test/clang-tidy/redundant-smartptr-get.cpp +++ b/clang-tools-extra/test/clang-tidy/readability-redundant-smartptr-get.cpp @@ -1,4 +1,4 @@ -// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s misc-redundant-smartptr-get %t +// RUN: $(dirname %s)/check_clang_tidy_fix.sh %s readability-redundant-smartptr-get %t // REQUIRES: shell #define NULL __null @@ -49,7 +49,7 @@ void Positive() { BarPtr u; // CHECK-FIXES: BarPtr u; BarPtr().get()->Do(); - // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: Redundant get() call on smart pointer. [misc-redundant-smartptr-get] + // CHECK-MESSAGES: :[[@LINE-1]]:3: warning: Redundant get() call on smart pointer. [readability-redundant-smartptr-get] // CHECK-MESSAGES: BarPtr().get()->Do(); // CHECK-FIXES: BarPtr()->Do(); |