[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-trivial] [Qemu-devel] [PATCH for-2.10] util/oslib-posix.c: Avo
From: |
Peter Maydell |
Subject: |
Re: [Qemu-trivial] [Qemu-devel] [PATCH for-2.10] util/oslib-posix.c: Avoid warning on NetBSD |
Date: |
Fri, 21 Jul 2017 11:22:59 +0100 |
On 20 July 2017 at 19:26, Eric Blake <address@hidden> wrote:
> On 07/20/2017 11:32 AM, Peter Maydell wrote:
>> On NetBSD the compiler warns:
>> util/oslib-posix.c: In function 'sigaction_invoke':
>> util/oslib-posix.c:589:5: warning: missing braces around initializer
>> [-Wmissing-braces]
>> siginfo_t si = { 0 };
>> ^
>
> Uggh. That is a broken compiler. C99 declares that 'anything = {0}' is
> supposed to be a valid way to zero-initialize anything.
>
>> util/oslib-posix.c:589:5: warning: (near initialization for 'si.si_pad')
>> [-Wmissing-braces]
>>
>> because on this platform siginfo_t is defined as
>> typedef union siginfo {
>> char si_pad[128]; /* Total size; for future expansion */
>> struct _ksiginfo _info;
>> } siginfo_t;
>>
>> Avoid this warning by initializing the struct with {} instead;
>> this is a GCC extension but we use it all over the codebase already.
>
> Well, I'm glad that works to shut up the broken compiler.
>
>>
>> Signed-off-by: Peter Maydell <address@hidden>
>> ---
>> util/oslib-posix.c | 2 +-
>> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> Reviewed-by: Eric Blake <address@hidden>
Thanks; applied to master.
-- PMM