FYI, comments on the codereview page. =] Thanks!<br><br><div class="gmail_quote">On Fri, Jul 15, 2011 at 5:06 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;">This patch improves how typo correction deals with overloaded functions, e.g. Sema::CorrectTypo now returns a set of Decls in the TypoCorrection if the lookup of a correction has a result kind of LookupResult::FoundOverloaded, and Sema::DiagnoseEmptyLookup has been modified to try to choose the best Decl from the set. With these changes, TypoCorrection is one step closer to never returning a non-keyword correction with a NULL Decl. I also ran memcheck using test/FixIt/typo-crash.cpp as a quick sanity check that I didn't accidentally introduce any memory leaks with the changes to the TypoCorrection class (though I think dynamically allocating the UnresolvedSet may be a bit of overkill and generally unnecessary--thoughts?).<br>

<br>The changes can also be reviewed at: <a href="http://codereview.appspot.com/4747047" target="_blank">http://codereview.appspot.com/4747047</a><br><br>Cheers,<br><font color="#888888">Kaelyn<br>
</font><br>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
<br></blockquote></div><br>