bison-patches
[Top][All Lists]
Advanced

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

Re: proposed namespace cleanup for bison.simple


From: Akim Demaille
Subject: Re: proposed namespace cleanup for bison.simple
Date: 30 Nov 2001 12:30:21 +0100
User-agent: Gnus/5.0808 (Gnus v5.8.8) XEmacs/21.4 (Civil Service)

| Bison's bison.simple file unnecessarily infringes on the user name
| space in several places.  It uses several local variables whose names
| do not begin with YY or yy, which can collide with user-defined
| macros.  It uses a couple of macros and functions that begin with _YY
| or __yy, which can collide with system names.  (Part of the problem is
| due to patches I've contributed over the years without fully
| understanding the name space issues -- sorry!)

Shame on you :)


Cute patch!  Just one note: we try as much as possible to move to #if
instead of #ifdef.  In particular, you should safely rely on #if
ERROR_VERBOSE and so on.


Please install on both.  If you have a problem with the trunk, you can
leave it to us.



reply via email to

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