<div dir="ltr">On Mon, Jul 15, 2013 at 4:25 PM, Samuel Benzaquen <span dir="ltr"><<a href="mailto:sbenza@google.com" target="_blank">sbenza@google.com</a>></span> wrote:<br><div class="gmail_extra"><div class="gmail_quote">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im"><br>
<br>
================<br>
Comment at: include/clang/ASTMatchers/ASTMatchersInternal.h:1441<br>
@@ +1440,3 @@<br>
+/// \code Getter<OuterT>::value() \endcode returns a<br>
+/// \code InnerTBase (OuterT::*)() \endcode, which is used to adapt a \c OuterT<br>
+/// object into a \c InnerT<br>
----------------<br>
</div><div class="im">Manuel Klimek wrote:<br>
> I think it's quite curious for the matchers that InnerTBase is returned by-value here (as that strongly limits what matchers this can be used with). I assume that was intentional?<br>
> I'm wondering whether there's a simpler solution hidden somewhere...<br>
</div>This class replaces the two classes that were being used for QualType and TypeLoc. Both of these are passed by value. I made it as generic as I needed for this specific change.<br></blockquote><div><br></div><div>Makes sense.</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div class="HOEnZb"><div class="h5"><br>
<br>
<a href="http://llvm-reviews.chandlerc.com/D1023" target="_blank">http://llvm-reviews.chandlerc.com/D1023</a><br>
<br>
BRANCH<br>
  type_traversal<br>
<br>
ARCANIST PROJECT<br>
  clang<br>
</div></div></blockquote></div><br></div></div>