bug-gnulib
[Top][All Lists]
Advanced

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

Re: [PATCH 2/2] Removed sc_prohibit_strcmp from the syntax-check


From: K Colford
Subject: Re: [PATCH 2/2] Removed sc_prohibit_strcmp from the syntax-check
Date: Wed, 21 May 2014 20:10:57 +0000

Then that's how I'll re implement it. 

-----Original Message-----
From: Eric Blake <address@hidden>
Date: Wed, 21 May 2014 14:08:50 
To: Paul Eggert<address@hidden>; Kieran Colford<address@hidden>; 
<address@hidden>
Subject: Re: [PATCH 2/2] Removed sc_prohibit_strcmp from the syntax-check

On 05/21/2014 01:58 PM, Paul Eggert wrote:
> On 05/21/2014 11:54 AM, Kieran Colford wrote:
>> I definitely agree
>> with your suggestion of having a module put them in config.h
> 
> Why config.h?  Can't it be put into a new include file, supplied by
> Gnulib?  stringops.h, say?We shouldn't bloat config.h unless we really
> need to.

Indeed, using a new header instead of bloating config.h ensures that
existing .c files in gnulib that already define their own STREQ won't
hit collisions (because they merely won't include the new file).

-- 
Eric Blake   eblake redhat com    +1-919-301-3266
Libvirt virtualization library http://libvirt.org



reply via email to

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