On Thu, Jul 2, 2015 at 12:47 AM, Manuel Klimek <kli...@google.com> wrote:
> Looks good. Thanks!
Thanks! Commit in r241256
~Aaron
>
> 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
