bison-patches
[Top][All Lists]
Advanced

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

Re: [tim-1] hash.h uses undefined macro __P()


From: Akim Demaille
Subject: Re: [tim-1] hash.h uses undefined macro __P()
Date: 11 Jan 2002 16:00:38 +0100
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Common Lisp)

| | The first of a couple of patches against CVS HEAD; mostly
| | in order to get it compiling properly on DJGPP.
| | 
| |lib/hash.h uses a macro (__P) for prototype support.  But
| |that macro is not defined.
| |This might be a glibc-induced problem (maybe glibc defines
| |__P?); if that is the case, it would probably be best to
| |rename the macro to __PROTO (as cvs does).
| |
| |2002-01-06  Tim Van Holder  <address@hidden>
| |
| |     * lib/hash.h (__P): Define this macro.
| 
| You've been granted CVS Write access.  Please, install.

I should have rejected this guy :(

__* is owned by the Libc, and we should not enter this pseudo
name space.  As a result, one Linux for instance:

gcc -DHAVE_CONFIG_H -I. -I. -I.. -I../intl -I.. -I../src -I../intl -I../lib    
-g -O2  -c `test -f hash.c || echo './'`hash.c
In file included from hash.c:22:
hash.h:25: warning: `__P' redefined
/usr/include/sys/cdefs.h:48: warning: this is the location of the previous 
definition


So could you adjust hash.c so that it uses PARAMS, like all the
others?  Thanks!



reply via email to

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