Cool. Let's not forget to land it :) -- Mathieu
2014-06-19 11:38 GMT-07:00 Alp Toker <[email protected]>: > > On 19/06/2014 19:48, Mathieu Baudet wrote: >> >> Hi Alp, >> >> Have you had a chance to look at the last patch I sent? > > > > Hi Mathieu, > > I see you're exercising the edge cases, I had to do a double take on this > one :-) > > It looks fine to me. > > Alp. > > > > >> >> -- Mathieu >> >> 2014-06-11 21:23 GMT-07:00 Mathieu Baudet <[email protected]>: >>> >>> Oups I thought I had tested but had not. Here is the real, and >>> hopefully final, patch. >>> >>> 2014-06-11 12:33 GMT-07:00 Mathieu Baudet <[email protected]>: >>>> >>>> Hi Alp, >>>> >>>> Sure. Here is another patch using the new syntax. >>>> >>>> -- Mathieu >>>> >>>> 2014-06-07 11:46 GMT-07:00 Alp Toker <[email protected]>: >>>>> >>>>> On 07/06/2014 03:44, Mathieu Baudet wrote: >>>>>> >>>>>> Hi, >>>>>> >>>>>> I found a corner case in ObjC++11 where the RecursiveASTVisitor could >>>>>> miss the body of a block. >>>>>> >>>>>> Please find the patch in attachment (unit test included). >>>>> >>>>> >>>>> The approach looks OK but please simplify with for (const auto &I : >>>>> D->captures()). >>>>> >>>>> Alp. >>>>> >>>>>> -- Mathieu >>>>> >>>>> >>>>> -- >>>>> http://www.nuanti.com >>>>> the browser experts >>>>> > > -- > http://www.nuanti.com > the browser experts > _______________________________________________ cfe-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits
