On Mar 29, 2013, at 9:30 AM, Loïc Jaquemet <[email protected]> wrote:

> any comments ?

I'll review it soon(ish).

> 
> 2013/3/25 Loïc Jaquemet <[email protected]>:
>>>> Does this handle anonymous structs/unions ? For example:
>>>> 
>>>> struct Test {
>>>>  struct {
>>>>    union {
>>>>      int foo;
>>>>    };
>>>>  };
>>>> };
>>>> 
>>>> If I pass "struct Test", "foo" will it work ? If yes, could you add a
>>>> related test ?
>>>> 
>>> 
>>> It does not work.
>>> I will look into that.
>> 
>> And now it does handle anonymous records.
>> I added some tests cases and changed the unit test code.
>> 
>> I also removed the duplicate clang_Cursor_getOffsetOf().
>> After consideration, it did not make sense, especially in the
>> anonymous record situation.
>> 
>> Please see attached diffs.
>> 
>> * Implementation of sizeof, alignof and offsetof for libclang.
>> * Unit Tests
>> * Python bindings
>> * Python tests
>> 
>> 
>> 
>> 
>> 
>> --
>> Loïc Jaquemet
> 
> 
> 
> -- 
> Loïc Jaquemet

_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to