diff options
author | Jim Meyering <meyering@redhat.com> | 2012-01-23 11:47:46 +0100 |
---|---|---|
committer | Jim Meyering <meyering@redhat.com> | 2012-01-23 13:52:21 +0100 |
commit | fb9eea88856d73a5f25295a52f6d4df7fabc565b (patch) | |
tree | f2a794d46cd67e923bb4d206aed79d8b17988b6e | |
parent | 219139f52178b3e020be1952c59b1ba76009c386 (diff) | |
download | bison-fb9eea88856d73a5f25295a52f6d4df7fabc565b.tar.gz |
build: avoid possibly-replaced fprintf in liby-source, yyerror.c
* lib/yyerror.c (yyerror): Use fputs and fputc rather than fprintf
with a mere "%s\n" format. Always return 0 now, on the assumption
that the return value was never used anyway.
Don't include <config.h> after all. This avoids a problem
reported by Thiru Ramakrishnan in
http://lists.gnu.org/archive/html/help-bison/2011-11/msg00000.html
* cfg.mk: Exempt lib/yyerror.c from the sc_require_config_h_first test.
* THANKS: Update.
-rw-r--r-- | THANKS | 1 | ||||
-rw-r--r-- | cfg.mk | 3 | ||||
-rw-r--r-- | lib/yyerror.c | 5 |
3 files changed, 6 insertions, 3 deletions
@@ -99,6 +99,7 @@ Sergei Steshenko sergstesh@yahoo.com Shura debil_urod@ngs.ru Steve Murphy murf@parsetree.com Summum Bonum sum@geekhouse.org +Thiru Ramakrishnan thiru.ramakrishnan@gmail.com Tim Josling tej@melbpc.org.au Tim Landscheidt tim@tim-landscheidt.de Tim Van Holder tim.van.holder@pandora.be @@ -55,4 +55,5 @@ update-copyright-env = \ UPDATE_COPYRIGHT_FORCE=1 UPDATE_COPYRIGHT_USE_INTERVALS=1 exclude_file_name_regexp--sc_space_tab = ^tests/(input|c\+\+)\.at$$ -exclude_file_name_regexp--sc_require_config_h_first = ^data/(glr|yacc)\.c$$ +exclude_file_name_regexp--sc_require_config_h_first = \ + ^(lib/yyerror|data/(glr|yacc))\.c$$ diff --git a/lib/yyerror.c b/lib/yyerror.c index 5eb339fa..c9f492f8 100644 --- a/lib/yyerror.c +++ b/lib/yyerror.c @@ -17,7 +17,6 @@ You should have received a copy of the GNU General Public License along with this program. If not, see <http://www.gnu.org/licenses/>. */ -#include <config.h> #include <stdio.h> int yyerror (char const *); @@ -25,5 +24,7 @@ int yyerror (char const *); int yyerror (char const *message) { - return fprintf (stderr, "%s\n", message); + fputs (message, stderr); + fputc ('\n', stderr); + return 0; } |