Please ignore this review request. I screwed up the patch upload AND the description (after returning from lunch, I'd forgotten I hadn't submitted the changes I meant to have reviewed to my local git repo yet).<div>
<br></div><div>Thanks,</div><div>Kaelyn<br><br><div class="gmail_quote">On Mon, Jan 23, 2012 at 12:39 PM,  <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">
Reviewers: <a href="http://kyrtzidis_apple.com" target="_blank">kyrtzidis_apple.com</a>,<br>
<br>
Description:<br>
Previously, for unqualified lookups, a positive cache hit is used as the<br>
only non-keyword correction and a negative cache hit immediately returns<br>
an empty TypoCorrection. With the new callback objects, this behavior<br>
causes false negatives by not accounting for the fact that callback<br>
objects alter the set of potential/allowed corrections. The new behavior<br>
is to seed the set of corrections with the cached correction (for<br>
positive hits) to estabilishing a baseline edit distance. Negative cache<br>
hits are only stored or used when either no callback object is provided<br>
or when it returns true for a call to ValidateCandidate with an empty<br>
TypoCorrection (i.e. when ValidateCandidate does not seem to be doing<br>
any checking of the TypoCorrection, such as when an instance of the base<br>
callback class is used solely to specify the set of keywords to be<br>
accepted).<br>
<br>
Please review this at <a href="http://codereview.appspot.com/5570046/" target="_blank">http://codereview.appspot.com/<u></u>5570046/</a><br>
<br>
Affected files:<br>
  M include/clang/Parse/Parser.h<br>
  M include/clang/Sema/Sema.h<br>
  M lib/Parse/ParseExpr.cpp<br>
  M lib/Parse/ParseExprCXX.cpp<br>
  M lib/Sema/SemaExpr.cpp<br>
  M lib/Sema/SemaLookup.cpp<br>
  M test/SemaCXX/typo-correction.<u></u>cpp<br>
<br>
<br>
</blockquote></div><br></div>