At Chandler's behest, I've uploaded this to Reitveld: <a href="http://codereview.appspot.com/1700049/show">http://codereview.appspot.com/1700049/show</a> . The latest patchset is very similar to the one attached to the previous email, mostly 80-col and comments, but one functional change to make TemplateArgumentLocHelper call into the TypeLoc traversal.<br>

<br>Nick<br><br><div class="gmail_quote">On 3 July 2010 18:58, Nick Lewycky <span dir="ltr"><<a href="mailto:nlewycky@google.com">nlewycky@google.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">

This patch adds basic traversal support for TypeLoc's to the RecursiveASTVisitor. The way it works is that the Visit*TypeLoc methods will call the matching Visit*Type methods, permitting users to continue using their Type-based visitors while permitting overrides of the TypeLoc specific methods. Many of the traversals have FIXME comments attached where the existing Type traversal does something that isn't captured in the matching TypeLoc, we fall back to traversing Types in that case.<br>


<br>In addition, 'TraverseDeclarationHelper' and 'TraverseClassTemplateSpecializationDecl' are modified to actually call into the TypeLoc versions instead of the Type versions.<br><br>Please review!<br><font color="#888888"><br>


Nick<br><br>
</font></blockquote></div><br>