qemu-devel
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Qemu-devel] [PATCH] build: disable -Wmissing-braces on older compil


From: Paolo Bonzini
Subject: Re: [Qemu-devel] [PATCH] build: disable -Wmissing-braces on older compilers
Date: Fri, 20 Oct 2017 12:37:20 +0200
User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.3.0

On 20/10/2017 12:27, Laszlo Ersek wrote:
>> +if compile_object "-Werror" "" ; then
>> +  :
>> +else
> Is this an established idiom for the configure script, in place of:
> 
>   if ! compile_object "-Werror" "" ; then
> 
> ?
> 
> Looks good to me otherwise.

I tend not use "if !".  In general that's just me, but in this case I
think it's useful to point out that a successful compile does nothing;
generally it's the successful compile that adjusts command line arguments.

Paolo



reply via email to

[Prev in Thread] Current Thread [Next in Thread]