<div class="gmail_quote">On Thu, Jul 28, 2011 at 2:25 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;">
<div id=":4ds">Here's an updated version of my patch to add function overload resolution to the typo correction, eliminating one of the ways that Sema::CorrectTypo returns a non-keyword correction without a corresponding NamedDecl. It incorporates all of the changes and feedback from <a href="http://codereview.appspot.com/4747047" target="_blank">http://codereview.appspot.com/4747047</a>. Given it's been almost two weeks since I sent out the first version and over a week since I made changes based on Chandler's last feedback (I was hoping he'd get a chance to look at those changes, but he's been swamped), I'd appreciate it of someone could give it a once-over and let me know whether it is ready for submission.<br>
</div></blockquote><div><br></div><div>I've made a few additional comments, but my primary concern with this patch is the representation of KeywordDecl. I think playing fast and loose with magical pointer values is going to come back to bite us. If we can't use 'NULL' as the magical pointer value, we should find some more robust way to represent a keyword in the correction results.</div>
<div><br></div><div>I'd certainly appreciate ideas from dgregor and any others on the list on the best way to represent that.</div><div><br></div><div>To make it easier for others to review, can you attach the latest patch to the email thread?</div>
</div>