summaryrefslogtreecommitdiffstats
path: root/clang/test/SemaCXX/warn-bool-conversion.cpp
Commit message (Collapse)AuthorAgeFilesLines
* Modify how the -verify flag works. Currently, the verification string andRichard Trieu2011-12-151-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* Correctly referring to the null pointer as 'null' not the macro 'NULL' in ↵David Blaikie2011-09-101-6/+6
| | | | | | the boolean conversion diagnostic message. llvm-svn: 139465
* Clean up the bool conversion warning. Group it with other conversionChandler Carruth2011-04-091-0/+24
warnings, and make its text appropriate for constant bool expressions other than 'false'. This should finish off PR9612. llvm-svn: 129205
OpenPOWER on IntegriCloud