patch: Fix RecursiveASTVisitor to visit types in ObjCPropertyDecl

Alp Toker alp at nuanti.com
Sun Jun 1 17:49:19 PDT 2014


On 02/06/2014 03:46, Mathieu Baudet wrote:
> Hi,
>
> Is there anything else I can do to help land this fix?

Hi Mathieu,

It was looking fine at first glance. Will take a closer look at it 
shorty if nobody beats me to it.

Thanks for holding

Alp.


>
> -- Mathieu
>
> 2014-05-22 10:15 GMT-07:00 Mathieu Baudet <mathieu.baudet at gmail.com>:
>> Thanks for your feedback. Here is an updated patch.
>>
>> -- Mathieu
>>
>> 2014-05-21 20:37 GMT-07:00 Jordan Rose <jordan_rose at apple.com>:
>>> This seems reasonable to me, but I'd appreciate it if someone more familiar with RAV took a look. Also, this should probably be applied to the DataRecursiveASTVisitor too, at least until the merge is complete.
>>>
>>> Jordan
>>>
>>>
>>> On May 20, 2014, at 18:59 , Mathieu Baudet <mathieu.baudet at gmail.com> wrote:
>>>
>>>> Hi,
>>>>
>>>> I made the patch in attachment.
>>>>
>>>> Hoping it is useful,
>>>> -- Mathieu
>>>> <clang_RecursiveASTVisitor_VisitsObjCPropertyType.patch>_______________________________________________
>>>> cfe-commits mailing list
>>>> cfe-commits at cs.uiuc.edu
>>>> http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

-- 
http://www.nuanti.com
the browser experts




More information about the cfe-commits mailing list