bug-bison
[Top][All Lists]
Advanced

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

Re: Bison-1.28 No Error Bug


From: Akim Demaille
Subject: Re: Bison-1.28 No Error Bug
Date: 03 Oct 2000 09:32:04 +0200
User-agent: Gnus/5.0807 (Gnus v5.8.7) XEmacs/21.1 (Channel Islands)

Please, don't drop the CC to the lists.

| The port is not very intrusive into the original source code, so it should
| (I hope) not be too difficlut to change that.
| 
| Here is a suggest for the code: Instead of having a lot of
| void warn PARAMS((char *));
| void warni PARAMS((char *, int));
| void warns PARAMS((char *, char *));
| void warnss PARAMS((char *, char *, char *));
| void warnsss PARAMS((char *, char *, char *, char *));
| you could write a single one
| 
| void warn (char *fmt, ...)
| {
|     va_list args;
| 
|     va_start(args, fmt);
|     warn_banner();
|     vfprintf(stderr, fmt, args);  */ Don't forget that little "v"! */
|     fputc('\n', stderr);
|     va_end(args);
| }
| 
| (I took it out of K&R, "CPL", p 174; I don't program C, just hack a little
| bit once in a while. :-) )

That's the approach advocated in the current beta.  It is still
unsatisfying: there are too many such functions (see complain.[ch]).

        Akim



reply via email to

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