<div class="gmail_quote">On Wed, Dec 14, 2011 at 4:38 PM, Richard Trieu <span dir="ltr"><<a href="mailto:rtrieu@google.com">rtrieu@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Author: rtrieu<br>
Date: Wed Dec 14 18:38:15 2011<br>
New Revision: 146619<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=146619&view=rev" target="_blank">http://llvm.org/viewvc/llvm-project?rev=146619&view=rev</a><br>
Log:<br>
Modify how the -verify flag works.  Currently, the verification string and<br>
diagnostic message are compared.  If either is a substring of the other, then<br>
no error is given.  This gives rise to an unexpected case:<br>
<br>
  // expect-error{{candidate function has different number of parameters}}<br>
<br>
will match the following error messages from Clang:<br>
<br>
  candidate function has different number of parameters (expected 1 but has 2)<br>
  candidate function has different number of parameters<br>
<br>
It will also match these other error messages:<br>
<br>
  candidate function<br>
  function has different number of parameters<br>
  number of parameters<br>
<br>
This patch will change so that the verification string must be a substring of<br>
the diagnostic message before accepting.  Also, all the failing tests from this<br>
change have been corrected.  Some stats from this cleanup:<br>
<br>
87 - removed extra spaces around verification strings<br>
70 - wording updates to diagnostics<br>
40 - extra leading or trailing characters (typos, unmatched parens or quotes)<br>
35 - diagnostic level was included (error:, warning:, or note:)<br>
18 - flag name put in the warning (-Wprotocol)<br></blockquote><div><br></div><div>Wow, that's scary. Thanks for fixing this!</div></div>