[libvirt] [PATCH python] define __GNUC_PREREQ macro before using it

Doug Goldstein cardoe at cardoe.com
Thu Jan 2 15:52:53 UTC 2014


On Tue, Dec 31, 2013 at 8:26 AM, Eric Blake <eblake at redhat.com> wrote:
> On 12/29/2013 03:52 PM, Doug Goldstein wrote:
>> We brought over use of the __GNUC_PREREQ macro from libvirt but didn't
>> bring over the definition of it. This brings over the macro from libvirt
>> sources.
>> ---
>>  libvirt-utils.h | 9 +++++++++
>>  1 file changed, 9 insertions(+)
>
> ACK (with #endif fixed).
>
>>
>> diff --git a/libvirt-utils.h b/libvirt-utils.h
>> index f55be7b..add7553 100644
>> --- a/libvirt-utils.h
>> +++ b/libvirt-utils.h
>> @@ -65,6 +65,15 @@
>>  # endif
>>
>>  # ifdef __GNUC__
>> +
>> +#  ifndef __GNUC_PREREQ
>> +#   if defined __GNUC__ && defined __GNUC_MINOR__
>> +#    define __GNUC_PREREQ(maj, min)                                        \
>> +    ((__GNUC__ << 16) + __GNUC_MINOR__ >= ((maj) << 16) + (min))
>> +#   else
>> +#    define __GNUC_PREREQ(maj,min) 0
>
> Copy and paste and all, but you could add a space after that comma for
> consistency.
>
>> +#   endif
>> +
>>  /**
>>   * ATTRIBUTE_UNUSED:
>>   *
>>
>
> --
> Eric Blake   eblake redhat com    +1-919-301-3266
> Libvirt virtualization library http://libvirt.org
>

Pushed with the fixes. Thanks all.

-- 
Doug Goldstein




More information about the libvir-list mailing list