[PATCH] D52688: [clang-tidy] NFC use CHECK-NOTES in tests for fuchsia-default-arguments
Jonas Toth via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Oct 3 03:30:33 PDT 2018
JonasToth updated this revision to Diff 168092.
JonasToth added a comment.
- remove FIX-IT as these will be filtered with the other patch
Repository:
rCTE Clang Tools Extra
https://reviews.llvm.org/D52688
Files:
test/clang-tidy/fuchsia-default-arguments.cpp
Index: test/clang-tidy/fuchsia-default-arguments.cpp
===================================================================
--- test/clang-tidy/fuchsia-default-arguments.cpp
+++ test/clang-tidy/fuchsia-default-arguments.cpp
@@ -1,14 +1,13 @@
// RUN: %check_clang_tidy %s fuchsia-default-arguments %t
int foo(int value = 5) { return value; }
-// CHECK-MESSAGES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+// CHECK-NOTES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: int foo(int value) { return value; }
int f() {
foo();
- // CHECK-MESSAGES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
- // CHECK-NEXT: note: default parameter was declared here:
- // CHECK-NEXT: int foo(int value = 5) { return value; }
+ // CHECK-NOTES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
+ // CHECK-NOTES: [[@LINE-8]]:9: note: default parameter was declared here
}
int bar(int value) { return value; }
@@ -21,16 +20,16 @@
class Baz {
public:
int a(int value = 5) { return value; }
- // CHECK-MESSAGES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+ // CHECK-NOTES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: int a(int value) { return value; }
int b(int value) { return value; }
};
class Foo {
// Fix should be suggested in declaration
int a(int value = 53);
- // CHECK-MESSAGES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+ // CHECK-NOTES: [[@LINE-1]]:9: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: int a(int value);
};
@@ -41,21 +40,21 @@
// Elided functions
void f(int = 5) {};
-// CHECK-MESSAGES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+// CHECK-NOTES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: void f(int) {};
void g(int) {};
// Should not suggest fix for macro-defined parameters
#define D(val) = val
void h(int i D(5));
-// CHECK-MESSAGES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+// CHECK-NOTES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES-NOT: void h(int i);
void x(int i);
void x(int i = 12);
-// CHECK-MESSAGES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+// CHECK-NOTES: [[@LINE-1]]:8: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: void x(int i);
void x(int i) {}
@@ -65,17 +64,17 @@
};
void S::x(int i = 12) {}
-// CHECK-MESSAGES: [[@LINE-1]]:11: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
+// CHECK-NOTES: [[@LINE-1]]:11: warning: declaring a parameter with a default argument is disallowed [fuchsia-default-arguments]
// CHECK-FIXES: void S::x(int i) {}
int main() {
S s;
s.x();
- // CHECK-MESSAGES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
- // CHECK-NEXT: note: default parameter was declared here:
+ // CHECK-NOTES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
+ // CHECK-NOTES: [[@LINE-9]]:11: note: default parameter was declared here
// CHECK-NEXT: void S::x(int i = 12) {}
x();
- // CHECK-MESSAGES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
- // CHECK-NEXT: note: default parameter was declared here:
+ // CHECK-NOTES: [[@LINE-1]]:3: warning: calling a function that uses a default argument is disallowed [fuchsia-default-arguments]
+ // CHECK-NOTES: [[@LINE-19]]:8: note: default parameter was declared here
// CHECK-NEXT: void x(int i = 12);
}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D52688.168092.patch
Type: text/x-patch
Size: 4450 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20181003/b639275e/attachment.bin>
More information about the cfe-commits
mailing list