Ok. Submitted as r160947.<div class="gmail_extra"><br><br><div class="gmail_quote">On Sun, Jul 29, 2012 at 11:27 AM, Manuel Klimek <span dir="ltr"><<a href="mailto:klimek@google.com" target="_blank">klimek@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 class="HOEnZb"><div class="h5">On Sun, Jul 29, 2012 at 10:41 AM, Daniel Jasper <<a href="mailto:djasper@google.com">djasper@google.com</a>> wrote:<br>

> Hi Manuel,<br>
><br>
> this fixes a problem of the ASTMatchFinder not visiting the VarDecls of a<br>
> function's parameters (see test case). I am not entirely sure it is the<br>
> right fix as I don't know what the original intention of overriding<br>
> TraverseTypeLoc was.<br>
<br>
</div></div>If I remember correctly, the original problem was that some types were<br>
not visited. With the fix it still looks like the type of the type loc<br>
is not traversed by itself... Not sure why we wouldn't want that in<br>
the RecursiveASTVisitor itself. cc'ing Richard, perhaps he has more<br>
insight here.<br>
<br>
Until we have that figured out, I think this is fine to go in, as it's<br>
a strict improvement.<br>
<br>
Cheers,<br>
/Manuel<br>
</blockquote></div><br></div>