I agree that a fixit for const mismatches would be nice too, but the focus of this patch is to address the situation where a function name is mistyped and there is some other mismatch (such as const) that would cause the typo-corrected function name to still not match an existing function declaration. Right now clang will correct the typo but not let the user know that there is still a problem (this is also not the only case where typo correction can expose/generate additional error messages).<br>
<br>Cheers,<br>Kaelyn<br><br><div class="gmail_quote">On Sat, Aug 20, 2011 at 10:34 PM, Jordy Rose <span dir="ltr"><<a href="mailto:jediknil@belkadan.com">jediknil@belkadan.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
I'm not a fan of emitting errors for code the user didn't write, but I'd be okay with it if there was a fixit for the presence or absence of 'const' as well. Then we could make a two-step recovery.<br>

<br>
Actually, a fixit for const mismatches is probably a good idea anyway...<br>
<br>
Jordy<br>
<div><div></div><div class="h5"><br>
<br>
On Aug 19, 2011, at 13:47, <a href="mailto:rikka@google.com">rikka@google.com</a> wrote:<br>
<br>
> Reviewers: chandlerc,<br>
><br>
> Description:<br>
> Not sure if this is the right way (without a major rewrite of<br>
> Sema::CorrectTypo and friends) to achieve the goal, but it solves the<br>
> issue brought up by Jordy in response to r137966 where an out-of-line<br>
> function definition is not rechecked against existing declarations to<br>
> see if it is valid after being typo-corrected.<br>
><br>
> Please review this at <a href="http://codereview.appspot.com/4929043/" target="_blank">http://codereview.appspot.com/4929043/</a><br>
><br>
> Affected files:<br>
>  M     lib/Sema/SemaDecl.cpp<br>
>  M     test/SemaCXX/function-redecl.cpp<br>
><br>
><br>
</div></div>> <issue_4929043_patch.diff>_______________________________________________<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>