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

Reply via email to