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.
This commit is contained in:
Jim Meyering
2012-01-23 11:47:46 +01:00
parent 219139f521
commit fb9eea8885
3 changed files with 6 additions and 3 deletions

1
THANKS
View File

@@ -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

3
cfg.mk
View File

@@ -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$$

View File

@@ -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;
}