From fb9eea88856d73a5f25295a52f6d4df7fabc565b Mon Sep 17 00:00:00 2001 From: Jim Meyering Date: Mon, 23 Jan 2012 11:47:46 +0100 Subject: [PATCH] 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 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. --- THANKS | 1 + cfg.mk | 3 ++- lib/yyerror.c | 5 +++-- 3 files changed, 6 insertions(+), 3 deletions(-) diff --git a/THANKS b/THANKS index 7603762d..7e246af0 100644 --- a/THANKS +++ b/THANKS @@ -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 diff --git a/cfg.mk b/cfg.mk index 2d4f1baf..6b3deb97 100644 --- a/cfg.mk +++ b/cfg.mk @@ -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 . */ -#include #include 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; } -- 2.45.2