I've addresses your comments and updated the patch. Thanks for reviewing it, Chandler! :)<br><br>I've also attached the updated patch that e.g. doesn't dynamically allocate the CorrectionDeclSet and doesn't maintain a separate flag for whether the correction is a keyword. I've also changed the KeywordDecl to be ~0x3 instead of -1 to avoid issues with data structures (such as UnresolvedSet that the first version of the patch used) assuming pointers are 64-bit aligned and manipulating the low 3 bits internally.<br>
<br>Cheers,<br>Kaelyn<br><br><div class="gmail_quote">On Fri, Jul 15, 2011 at 6:34 PM, Chandler Carruth <span dir="ltr"><<a href="mailto:chandlerc@google.com">chandlerc@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, comments on the codereview page. =] Thanks!<br><br><div class="gmail_quote"><div><div></div><div class="h5">On Fri, Jul 15, 2011 at 5:06 PM, Kaelyn Uhrain <span dir="ltr"><<a href="mailto:rikka@google.com" target="_blank">rikka@google.com</a>></span> wrote:<br>

</div></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div><div></div><div class="h5">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></div></div>_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" target="_blank">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>
</blockquote></div><br>