Re: [PATCH] maint: turn on gcc logical-op checking

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

 



2010/8/27 Eric Blake <eblake@xxxxxxxxxx>:
> On 07/26/2010 08:27 AM, Eric Blake wrote:
>>
>> On 07/24/2010 02:36 PM, Matthias Bolte wrote:
>>>>
>>>> + * nuking the wrappers; even if it removes some type-check safety.  */
>>>> +# undef curl_easy_getinfo
>>>> +# undef curl_easy_setopt
>>>> +
>>>>  #define VIR_FROM_THIS VIR_FROM_ESX
>>>
>>> No need to hack here. We can define CURL_DISABLE_TYPECHECK to disable
>>> those type checks.
>>
>> Yes, that sounds nicer.
>
> Older gcc versions had a bug that meant the warnings in the libcurl header
> were spurious, since corrected in newer gcc, available in Rawhide.  In the
> meantime, a newer libcurl release now available on Fedora 13 works around
> the older gcc bug.  So, either way, should we remove CURL_DISABLE_TYPECHECK
> (effectively re-enabling the type safety), now that the spurious warnings
> are no longer triggered?
>
> https://bugzilla.redhat.com/show_bug.cgi?id=617757
>

Well, even if this is fixed in GCC >= 4.5.0 and a workaround was added
to libcurl and Fedora isn't affected anymore doesn't mean that this
problem is fixed for all systems/distros out there.

I think we could remove CURL_DISABLE_TYPECHECK for GCC >= 4.5.0, but
keep it for older GCC versions.

I tried to figure out how to detect GCC version in configure.ac, but
my autotools-fu is weak today.

Matthias

--
libvir-list mailing list
libvir-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/libvir-list



[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]