Since trying to refactor Sema::ActOnFunctionDeclarator has proved to be far more time consuming and invasive (lots of state needed throughout the func spread among multiple variables with far too many order-dependent calls to other complex functions that depend heavily on side-effects), I've updated the current fix for DiganoseInvalidRedeclaration typo correction to pack the extra args needed to call Sema::ActOnFunctionDeclarator into a struct. And yes, I suspect the run-on sentence complete with tangents is a symptom of spending so much time digging around in and trying to untangle Sema::ActOnFunctionDeclarator.<br>
<br>The updated patch is attached, as rietveld is still too lame to send out an email with a patch or any kind of description when using its upload.py script to update an existing issue with a new patchset.<br><br>Cheers,<br>
Kaelyn<br><br><div class="gmail_quote">On Tue, Aug 30, 2011 at 4:16 PM,  <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><div></div><div class="h5"><a href="http://codereview.appspot.com/4929043/" target="_blank">http://codereview.appspot.com/<u></u>4929043/</a><br>
</div></div></blockquote></div><br>