[PATCH] AST matcher for exception variables

Manuel Klimek klimek at google.com
Wed Jul 15 10:01:10 PDT 2015


Ah, sorry for the delay (you could help me by using phab - but no worries,
small changes like this are easy enough).

I'd name the thing isExceptionVariable, as that's also the method we're
calling on the AST.
Apart from that LGTM

On Wed, Jul 15, 2015 at 6:34 PM Aaron Ballman <aaron at aaronballman.com>
wrote:

> Ping
>
> ~Aaron
>
> On Fri, Jul 10, 2015 at 3:15 PM, Aaron Ballman <aaron at aaronballman.com>
> wrote:
> > There is not currently a way to match against a variable declaration
> > that is an exception variable. For instance:
> >
> > void f() {
> >   try {
> >     int X;
> >   } catch (int Y) {
> >   }
> > }
> >
> > You can match int X and int Y, but you cannot currently match only int
> > Y. The attached patch adds a matcher to allow that, as well as test
> > code and documentation.
> >
> > ~Aaron
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20150715/fcb77a0d/attachment.html>


More information about the cfe-commits mailing list