Ping? I'd like at least a little feedback before submitting this patch since it is the first time I've done anything with the parser code.<br><br><div class="gmail_quote">On Tue, Sep 13, 2011 at 5:22 PM, Kaelyn Uhrain <span dir="ltr"><<a href="mailto:rikka@google.com">rikka@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">FYI, this patch allows for typo correction in what is from a user perspective a reasonable case where clang should suggest a correction but does not. Right now in:<br>
<br>namespace test {<br> class FizBinHelper {};<br>}<br>
<br>void foo() {<br> test::FizBinHelper *x;<br> x = new test::FizbinHelper;<br>}<br><br>clang will just say "expected a type" instead of suggesting correcting test::FizbinHelper to test::FizBinHelper (fixing a simple case-mismatch typo).<br>

<br>Just in case it wasn't clear what the context of the change is from the changes to the test cases. ;)<br><br>Cheers,<br><font color="#888888">Kaelyn</font><div><div></div><div class="h5"><br><br><div class="gmail_quote">
On Tue, Sep 13, 2011 at 5:16 PM,  <span dir="ltr"><<a href="mailto:rikka@google.com" target="_blank">rikka@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Reviewers: <a href="http://cfe-commits_cs.uiuc.edu" target="_blank">cfe-commits_cs.uiuc.edu</a>,<br>
<br>
<br>
<br>
Please review this at <a href="http://codereview.appspot.com/5014044/" target="_blank">http://codereview.appspot.com/<u></u>5014044/</a><br>
<br>
Affected files:<br>
  M include/clang/Parse/Parser.h<br>
  M include/clang/Sema/Sema.h<br>
  M lib/Parse/ParseDecl.cpp<br>
  M lib/Parse/ParseExprCXX.cpp<br>
  M lib/Parse/Parser.cpp<br>
  M lib/Sema/SemaDecl.cpp<br>
  M test/SemaCXX/missing-<u></u>namespace-qualifier-typo-<u></u>corrections.cpp<br>
<br>
<br>
</blockquote></div><br>
</div></div></blockquote></div><br>