diff options
author | Alexander Kornienko <alexfh@google.com> | 2014-10-26 02:58:07 +0000 |
---|---|---|
committer | Alexander Kornienko <alexfh@google.com> | 2014-10-26 02:58:07 +0000 |
commit | 106c8e0898688fbdff96b017828da9762f1d6b20 (patch) | |
tree | fffad8ed33e1c3b61cc356c645434bbfe2465afd /clang-tools-extra/test/clang-tidy/google-runtime-int.cpp | |
parent | 6775ce3ad3e9bf3e2598143b7cac4b5ecdaafa93 (diff) | |
download | bcm5719-llvm-106c8e0898688fbdff96b017828da9762f1d6b20.tar.gz bcm5719-llvm-106c8e0898688fbdff96b017828da9762f1d6b20.zip |
[clang-tidy] check_clang_tidy_fix.sh -> check_clang_tidy.sh
Summary: Make the script suitable for checking just messages. Move most of the tests to use it. Clean up the tests: shorten messages, insert line numbers, remove unnecessary RUN: lines, etc.
Reviewers: klimek
Reviewed By: klimek
Subscribers: curdeius, cfe-commits
Differential Revision: http://reviews.llvm.org/D5989
llvm-svn: 220634
Diffstat (limited to 'clang-tools-extra/test/clang-tidy/google-runtime-int.cpp')
-rw-r--r-- | clang-tools-extra/test/clang-tidy/google-runtime-int.cpp | 39 |
1 files changed, 20 insertions, 19 deletions
diff --git a/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp b/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp index 44d1b650272..3a344b8097a 100644 --- a/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp +++ b/clang-tools-extra/test/clang-tidy/google-runtime-int.cpp @@ -1,13 +1,14 @@ -// RUN: clang-tidy -checks=-*,google-runtime-int %s -- -x c++ 2>&1 | FileCheck %s -implicit-check-not='{{warning:|error:}}' +// RUN: $(dirname %s)/check_clang_tidy.sh %s google-runtime-int %t +// REQUIRES: shell long a(); -// CHECK: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int{{..}}' +// CHECK-MESSAGES: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int{{..}}' typedef unsigned long long uint64; // NOLINT long b(long = 1); -// CHECK: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int{{..}}' -// CHECK: [[@LINE-2]]:8: warning: consider replacing 'long' with 'int{{..}}' +// CHECK-MESSAGES: [[@LINE-1]]:1: warning: consider replacing 'long' with 'int{{..}}' +// CHECK-MESSAGES: [[@LINE-2]]:8: warning: consider replacing 'long' with 'int{{..}}' template <typename T> void tmpl() { @@ -15,45 +16,45 @@ void tmpl() { } short bar(const short, unsigned short) { -// CHECK: [[@LINE-1]]:1: warning: consider replacing 'short' with 'int16' -// CHECK: [[@LINE-2]]:17: warning: consider replacing 'short' with 'int16' -// CHECK: [[@LINE-3]]:24: warning: consider replacing 'unsigned short' with 'uint16' +// CHECK-MESSAGES: [[@LINE-1]]:1: warning: consider replacing 'short' with 'int16' +// CHECK-MESSAGES: [[@LINE-2]]:17: warning: consider replacing 'short' with 'int16' +// CHECK-MESSAGES: [[@LINE-3]]:24: warning: consider replacing 'unsigned short' with 'uint16' long double foo = 42; uint64 qux = 42; unsigned short port; const unsigned short bar = 0; -// CHECK: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' +// CHECK-MESSAGES: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' long long *baar; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' const unsigned short &bara = bar; -// CHECK: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' +// CHECK-MESSAGES: [[@LINE-1]]:9: warning: consider replacing 'unsigned short' with 'uint16' long const long moo = 1; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' long volatile long wat = 42; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'long long' with 'int64' unsigned long y; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long' with 'uint{{..}}' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'unsigned long' with 'uint{{..}}' unsigned long long **const *tmp; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' unsigned long long **const *&z = tmp; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'unsigned long long' with 'uint64' unsigned short porthole; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'unsigned short' with 'uint16' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'unsigned short' with 'uint16' uint64 cast = (short)42; -// CHECK: [[@LINE-1]]:18: warning: consider replacing 'short' with 'int16' +// CHECK-MESSAGES: [[@LINE-1]]:18: warning: consider replacing 'short' with 'int16' #define l long l x; tmpl<short>(); -// CHECK: [[@LINE-1]]:8: warning: consider replacing 'short' with 'int16' +// CHECK-MESSAGES: [[@LINE-1]]:8: warning: consider replacing 'short' with 'int16' } void p(unsigned short port); void qux() { short port; -// CHECK: [[@LINE-1]]:3: warning: consider replacing 'short' with 'int16' +// CHECK-MESSAGES: [[@LINE-1]]:3: warning: consider replacing 'short' with 'int16' } |