Looks good. Thanks!

On Wed, Jul 1, 2015 at 11:16 PM Aaron Ballman <aa...@aaronballman.com>
wrote:

> I was making a brief foray into AST matchers over exception handling
> code today, and I noticed there's no way (that I could find) to match
> only catch-all exception handlers. Eg)
>
> try {
> } catch (int) {
> } catch (...) { // Only want to match this
> }
>
> The attached patch adds the matcher, test case, and documentation for
> such a thing. I am not strongly tied to "catch all" as the name, but
> alternatives like "handlesAnyType" sound like it would match
> catch(int) as well as catch(...). If there's a better name, I'd be
> happy to make use of it.
>
> Thanks!
>
> ~Aaron
>
_______________________________________________
cfe-commits mailing list
cfe-commits@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to