diff options
| author | Richard Trieu <rtrieu@google.com> | 2011-12-15 00:38:15 +0000 |
|---|---|---|
| committer | Richard Trieu <rtrieu@google.com> | 2011-12-15 00:38:15 +0000 |
| commit | 553b2b2e5dde2839618edbc7b60f6c0677774853 (patch) | |
| tree | 939113f97b1097c1629ec5d75d3b84be7e3b6582 /clang/test/SemaObjC/conditional-expr.m | |
| parent | 0b144e160a3129708d1494e88d614cdce2867160 (diff) | |
| download | bcm5719-llvm-553b2b2e5dde2839618edbc7b60f6c0677774853.tar.gz bcm5719-llvm-553b2b2e5dde2839618edbc7b60f6c0677774853.zip | |
Modify how the -verify flag works. Currently, the verification string and
diagnostic message are compared. If either is a substring of the other, then
no error is given. This gives rise to an unexpected case:
// expect-error{{candidate function has different number of parameters}}
will match the following error messages from Clang:
candidate function has different number of parameters (expected 1 but has 2)
candidate function has different number of parameters
It will also match these other error messages:
candidate function
function has different number of parameters
number of parameters
This patch will change so that the verification string must be a substring of
the diagnostic message before accepting. Also, all the failing tests from this
change have been corrected. Some stats from this cleanup:
87 - removed extra spaces around verification strings
70 - wording updates to diagnostics
40 - extra leading or trailing characters (typos, unmatched parens or quotes)
35 - diagnostic level was included (error:, warning:, or note:)
18 - flag name put in the warning (-Wprotocol)
llvm-svn: 146619
Diffstat (limited to 'clang/test/SemaObjC/conditional-expr.m')
| -rw-r--r-- | clang/test/SemaObjC/conditional-expr.m | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/clang/test/SemaObjC/conditional-expr.m b/clang/test/SemaObjC/conditional-expr.m index 74ab59bdb9e..f97224fe7af 100644 --- a/clang/test/SemaObjC/conditional-expr.m +++ b/clang/test/SemaObjC/conditional-expr.m @@ -21,7 +21,7 @@ @end @interface DTFilterOutputStream2 -- nextOutputStream; // expected-note {{{{method definition for 'nextOutputStream' not found}} +- nextOutputStream; // expected-note {{method definition for 'nextOutputStream' not found}} @end @implementation DTFilterOutputStream2 // expected-warning {{incomplete implementation}} @@ -100,7 +100,7 @@ void f8(int a, A<P0> *x, A *y) { } void f9(int a, A<P0> *x, A<P1> *y) { - id l0 = (a ? x : y ); // expected-warning {{incompatible operand types ('A<P0> *' and 'A<P1> *')'}} + id l0 = (a ? x : y ); // expected-warning {{incompatible operand types ('A<P0> *' and 'A<P1> *')}} A<P0> *l1 = (a ? x : y ); // expected-warning {{incompatible operand types ('A<P0> *' and 'A<P1> *')}} A<P1> *l2 = (a ? x : y ); // expected-warning {{incompatible operand types ('A<P0> *' and 'A<P1> *')}} [ (a ? x : y ) intProp ]; // expected-warning {{incompatible operand types ('A<P0> *' and 'A<P1> *')}} |

