[cfe-commits] [PATCH][MS][Review request] - Microsoft C anonymous struct, second try

[Francois Pichet]

ping

Anybody planning to review that?
Or should I go ahead and checkin?

On Sun, Oct 24, 2010 at 9:39 PM, Francois Pichet <pichet2000 at gmail.com> wrote:
> Hi
>
> This is an updated patch for the Microsoft C anonymous struct
> extensions. (PR 6829)
> MSDN documentation: http://msdn.microsoft.com/en-us/library/z2cx9y4f.aspx
>
> My first patch didn't deal with anonymous struct defined via typedef.
> for example:
> typedef struct { int a} A;
> struct B {A; int b;};
>
> I struggled and I finally decided to deal that case by inserting an
> implicit RecordDecl to make the handling uniform. (To have
> isTransparentContext return true)
> An alternative solution would be to have isTransparentContext() deal
> with a sole typedef name as a transparent context. But I found that
> solution intrusive for such an obscure MS C extension.
>
> Please review.
>