bison-patches
[Top][All Lists]
Advanced

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

Re: Preparing 3.4.2


From: Akim Demaille
Subject: Re: Preparing 3.4.2
Date: Sun, 8 Sep 2019 11:42:59 +0200

I have removed the following commit, as it changes the set of translated 
strings (and I don't want to have to release a beta of 3.4.2 to leave time to 
the translators to update the translations).

> commit 9c1480467664577982f753414d56c84b67352737
> Author: Akim Demaille <address@hidden>
> Date:   Sat Jul 6 09:09:51 2019 +0200
> 
>    doc: fix inaccuracies wrt --define and --force-define
> 
>    The doc says that -Dfoo=bar is the same as %define foo "bar".  It is
>    not: the quotes are not added (and it makes a difference).
> 
>    * doc/bison.texi (Tuning the Parser): Fix the definition of -D/-F
>    * src/getargs.c (usage): Likewise.

I have added the following one, to please the CI:

> commit d3a86f7b20bd3a6756f2382fad12beafba0443e7
> Author: Akim Demaille <address@hidden>
> Date:   Fri Sep 6 07:21:59 2019 +0200
> 
>     yacc.c: beware of GCC's -Wmaybe-uninitialized
>     
>     Test 400 (calc.at:773: testing Calculator api.push-pull=both
>     api.pure=full parse.error=verbose %debug %locations %defines
>     api.prefix={calc} %verbose %yacc) fails on the CI with GCC 8 on
>     Bionic:
>     
>         400. calc.at:773: testing Calculator api.push-pull=both api.pure=full 
> parse.error=verbose %debug %locations %defines api.prefix={calc} %verbose
>  %yacc  ...
>         ../../tests/calc.at:773: bison --color=no -fno-caret -Wno-deprecated 
> -o calc.c calc.y
>         ../../tests/calc.at:773: $CC $CFLAGS $CPPFLAGS  $LDFLAGS -o calc 
> calc.c calc-lex.c calc-main.c $LIBS
>         stderr:
>         calc.y: In function 'int calcpush_parse(calcpstate*, int, const 
> CALCSTYPE*, CALCLTYPE*)':
>         calc.y:26:20: error: 'yylval.CALCSTYPE::ival' may be used 
> uninitialized in this function [-Werror=maybe-uninitialized]
>          %printer { fprintf (yyo, "%d", $$); } <ival>;
>                             ^
>         calc.c:1272:9: note: 'yylval.CALCSTYPE::ival' was declared here
>          YYSTYPE yylval YY_INITIAL_VALUE (= yyval_default);
>                  ^~~~~~
>         cc1plus: all warnings being treated as errors
>         stdout:
>         ../../tests/calc.at:773: exit code was 1, expected 0
>         400. calc.at:773: 400. Calculator api.push-pull=both api.pure=full 
> parse.error=verbose %debug %locations %defines api.prefix={calc} %verbose %y
> acc  (calc.at:773): FAILED (calc.at:773)
>     
>     * data/skeletons/c.m4 (yy_symbol_value_print): Disable the warning
>     locally.


and updated NEWS to:

** Bug fixes

  In some cases, when warnings are disabled, bison could emit tons of white
  spaces as diagnostics.

  When running out of memory, bison could crash (found by fuzzing).

  When defining twice the EOF token, bison would crash.

  New warnings from recent compilers have been addressed in the generated
  parsers (yacc.c, glr.c, glr.cc).




reply via email to

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