<div dir="ltr">Looks good. Thanks!</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Jul 1, 2015 at 11:16 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">I was making a brief foray into AST matchers over exception handling<br>
code today, and I noticed there's no way (that I could find) to match<br>
only catch-all exception handlers. Eg)<br>
<br>
try {<br>
} catch (int) {<br>
} catch (...) { // Only want to match this<br>
}<br>
<br>
The attached patch adds the matcher, test case, and documentation for<br>
such a thing. I am not strongly tied to "catch all" as the name, but<br>
alternatives like "handlesAnyType" sound like it would match<br>
catch(int) as well as catch(...). If there's a better name, I'd be<br>
happy to make use of it.<br>
<br>
Thanks!<br>
<br>
~Aaron<br>
</blockquote></div>