This revision was automatically updated to reflect the committed changes. Closed by commit rL292926: [clang-tidy] Fix NOLINT test (authored by alexfh).
Changed prior to commit: https://reviews.llvm.org/D26466?vs=77378&id=85571#toc Repository: rL LLVM https://reviews.llvm.org/D26466 Files: clang-tools-extra/trunk/test/clang-tidy/nolint.cpp Index: clang-tools-extra/trunk/test/clang-tidy/nolint.cpp =================================================================== --- clang-tools-extra/trunk/test/clang-tidy/nolint.cpp +++ clang-tools-extra/trunk/test/clang-tidy/nolint.cpp @@ -1,5 +1,12 @@ // RUN: %check_clang_tidy %s google-explicit-constructor,clang-diagnostic-unused-variable,clang-analyzer-core.UndefinedBinaryOperatorResult %t -- -extra-arg=-Wunused-variable -- -I%S/Inputs/nolint +#include "trigger_warning.h" +void I(int& Out) { + int In; + A1(In, Out); +} +// CHECK-MESSAGES-NOT: trigger_warning.h:{{.*}} warning +// CHECK-MESSAGES-NOT: :[[@LINE-4]]:{{.*}} note class A { A(int i); }; // CHECK-MESSAGES: :[[@LINE-1]]:11: warning: single-argument constructors must be marked explicit @@ -28,12 +35,4 @@ #define DOUBLE_MACRO MACRO(H) // NOLINT DOUBLE_MACRO -#include "trigger_warning.h" -void I(int& Out) { - int In; - A1(In, Out); -} -// CHECK-NOT: trigger_warning.h:{{.*}} warning: The left operand of '>' is a garbage value -// CHECK-NOT: :[[@LINE-4]]:{{.*}} note - // CHECK-MESSAGES: Suppressed 8 warnings (8 NOLINT)
Index: clang-tools-extra/trunk/test/clang-tidy/nolint.cpp =================================================================== --- clang-tools-extra/trunk/test/clang-tidy/nolint.cpp +++ clang-tools-extra/trunk/test/clang-tidy/nolint.cpp @@ -1,5 +1,12 @@ // RUN: %check_clang_tidy %s google-explicit-constructor,clang-diagnostic-unused-variable,clang-analyzer-core.UndefinedBinaryOperatorResult %t -- -extra-arg=-Wunused-variable -- -I%S/Inputs/nolint +#include "trigger_warning.h" +void I(int& Out) { + int In; + A1(In, Out); +} +// CHECK-MESSAGES-NOT: trigger_warning.h:{{.*}} warning +// CHECK-MESSAGES-NOT: :[[@LINE-4]]:{{.*}} note class A { A(int i); }; // CHECK-MESSAGES: :[[@LINE-1]]:11: warning: single-argument constructors must be marked explicit @@ -28,12 +35,4 @@ #define DOUBLE_MACRO MACRO(H) // NOLINT DOUBLE_MACRO -#include "trigger_warning.h" -void I(int& Out) { - int In; - A1(In, Out); -} -// CHECK-NOT: trigger_warning.h:{{.*}} warning: The left operand of '>' is a garbage value -// CHECK-NOT: :[[@LINE-4]]:{{.*}} note - // CHECK-MESSAGES: Suppressed 8 warnings (8 NOLINT)
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits