Looks like the indenting on the FIXME you added needs to be fixed so that the start of both lines of the comment are equal to each other and to the surrounding code, but otherwise the code changes look fine. One big issue regarding the unit test though:  the test does not test clang's "-fixit" code rewrite functionality so should not be under test/FixIt/; test/SemaCXX/ is the appropriate place for this C++ unit test. Naming the file something a bit more descriptive than just "typo-crash.cpp", such as "function-overload-typo-crash.cpp", would also be nice.<br>
<br>Cheers,<br>Kaelyn<br><br><div class="gmail_quote">On Sat, Jul 9, 2011 at 1:33 PM, Hans Wennborg <span dir="ltr"><<a href="mailto:hans@chromium.org">hans@chromium.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
Don't crash when typo correction finds an overloaded or template function.<br>
<br>
It would be cool if we could do overload resolution to suggest the<br>
right function here, but this at least fixes the crashing (PR10283).<br>
<br>
Ok to commit?<br>
<br>
Thanks,<br>
<font color="#888888">Hans<br>
</font></blockquote></div><br>