This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

RE: Remove parameter names from libiberty.h


----Original Message----
>From: H. J. Lu
>Sent: 14 April 2005 18:18

> On Thu, Apr 14, 2005 at 12:17:11PM -0400, Andrew Pinski wrote:
>>> 
>>>> Kaveh R. Ghazi writes:
>>>> As noted here:
>>>> http://sourceware.org/ml/binutils/2005-04/msg00269.html
>>>> 
>>>> The new f*open_unlocked function prototypes in libiberty.h are in
>>>> conflict with binutils sources because one of the parameter names
>>>> ("mode") gets defined to "31" and the build dies.
>>> 
>>> Why not alternatively fix the real problem (as you had noted), as
>>> opposed to indirectly establishing the policy that libiberty prototypes
>>> don't include parameter names (as it doesn't seem like the true
>>> solution to the problem). 
>> 
>> I don't know why I am replying to you but ...
>> Anyways there is no other way to fix the problem correctly.  If we change
>> the parameter name, someone else might have defined it so we get into
>> a cycle.
>> 
> 
> There are many prototypes in libiberty.h with names.
> 


  My two cents is that it's asking for trouble to #define an all-lower case
word, particularly an extremely common one such as 'mode'.  I had a quick
grep but couldn't figure out where it's coming from.  I guess that sort of
thing is vaguely-ok-if-innately-fragile if it's being defined by the
m88k-dis source file, but if there's a public header in either binutils or
libiberty that's doing it, I'd think it ought to change.



    cheers,
      DaveK
-- 
Can't think of a witty .sigline today....


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]