[MPlayer-cvslog] r33911 - trunk/stream/cache2.c

[Ivan Kalvachev]

On 7/25/11, Nicolas George <nicolas.george at normalesup.org> wrote:
> Le septidi 7 thermidor, an CCXIX, Reimar Döffinger a écrit :
>> > +    struct sigaction sa = { { 0 } };
>> This is wrong, POSIX does not require it to start with or even contain
>> a struct or union.
>> Thus your code potentially will not compile on some systems.
>> If you want to get rid of the warning you'll have to use memset.
>
> What about this:
>
> struct sigaction sa = { .sa_flags = 0 };
>
> ? sa_flags is required to be present and an int.
>

I'd prefer the memset.
KiSS