<div dir="ltr">Ah, sorry for the delay (you could help me by using phab - but no worries, small changes like this are easy enough).<div><br></div><div>I'd name the thing isExceptionVariable, as that's also the method we're calling on the AST.</div><div>Apart from that LGTM</div></div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 15, 2015 at 6:34 PM Aaron Ballman <<a href="mailto:aaron@aaronballman.com">aaron@aaronballman.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping<br>
<br>
~Aaron<br>
<br>
On Fri, Jul 10, 2015 at 3:15 PM, Aaron Ballman <<a href="mailto:aaron@aaronballman.com" target="_blank">aaron@aaronballman.com</a>> wrote:<br>
> There is not currently a way to match against a variable declaration<br>
> that is an exception variable. For instance:<br>
><br>
> void f() {<br>
>   try {<br>
>     int X;<br>
>   } catch (int Y) {<br>
>   }<br>
> }<br>
><br>
> You can match int X and int Y, but you cannot currently match only int<br>
> Y. The attached patch adds a matcher to allow that, as well as test<br>
> code and documentation.<br>
><br>
> ~Aaron<br>
</blockquote></div>