Fix RecursiveASTVisitor to visit copy expressions of captured variables in blocks (ObjC++11)

[Mathieu Baudet]

Cool. Let's not forget to land it :)
--
Mathieu

2014-06-19 11:38 GMT-07:00 Alp Toker <alp at nuanti.com>:
>
> 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 <mathieu.baudet at gmail.com>:
>>>
>>> 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 <mathieu.baudet at gmail.com>:
>>>>
>>>> Hi Alp,
>>>>
>>>> Sure. Here is another patch using the new syntax.
>>>>
>>>> -- Mathieu
>>>>
>>>> 2014-06-07 11:46 GMT-07:00 Alp Toker <alp at nuanti.com>:
>>>>>
>>>>> 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
>