]> git.saurik.com Git - bison.git/blobdiff - ChangeLog
The location tracking mechanism is precious for parse error
[bison.git] / ChangeLog
index c6550632195f4d4e91cc9fdbda2866e388c443f5..4ddc4903de5fbba6ac4708135c49077162603c14 100644 (file)
--- a/ChangeLog
+++ b/ChangeLog
@@ -1,3 +1,215 @@
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       The location tracking mechanism is precious for parse error
+       messages.  Nevertheless, it is enabled only when `@n' is used in
+       the grammar, which is a different issue (you can use it in error
+       message, but not in the grammar per se).  Therefore, there should
+       be another means to enable it.
+
+       * src/getargs.c (getargs): Support `--locations'.
+       (usage): Report it.
+       * src/getargs.h (locationsflag): Export it.
+       * src/lex.c (percent_table): Support `%locations'.
+       * src/reader.c (yylsp_needed): Remove this variable, now replaced
+       with `locationsflag'.
+       * doc/bison.texinfo: Document `--locations' and `%locations'.
+       Sort the options.
+       * tests/calc.m4: Test it.
+
+       For regularity of the names, replace each
+       (nolineflag, toknumflag, rawtokenumflag, noparserflag): with...
+       (no_lineflag, token_tableflag, rawflag, no_parserflag): this.
+       In addition replace each `flag' with `_flag'.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       Also test parse error messages, including with YYERROR_VERBOSE.
+
+       * tests/calc.m4 (calc.y): Add support for `exp = exp' (non
+       associative).
+       Use it to check the computations.
+       Use it to check `nonassoc' is honored.
+       (AT_DATA_CALC_Y): Equip `calc.y' with YYERROR_VERBOSE when passed
+       `--yyerror-verbose'.
+       (_AT_CHECK_CALC): Adjust to this option.
+       (_AT_CHECK_CALC_ERROR): New macro to check parse error messages.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       Test also `--verbose', `--defines' and `--name-prefix'.  Testing
+       the latter demonstrates a flaw in the handling of non debugging
+       parsers introduced by myself on 2000-03-16: `#define yydebug 0'
+       was used in order to simplify:
+
+       #if YYDEBUG
+       if (yydebug)
+       {
+       ...
+       }
+       #endif
+
+       into
+
+       if (yydebug)
+       {
+       ...
+       }
+
+       unfortunately this leads to a CPP conflict when
+       `--name-prefix=foo' is used since it produces `#define yydebug
+       foodebug'.
+
+       * src/bison.s1 [!YYDEBUG]: Do not define yydebug.
+       (YYDPRINTF): New macro.
+       Spread its use.
+       * tests/calc.m4 (AT_CHECK_CALC): Do require a title, build it from
+       the bison options.
+       Also test `--verbose', `--defines' and `--name-prefix'.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       Improve the readability of the produced parsers.
+
+       * src/bison.s1: Formatting changes.
+       Improve the comment related to the `$' mark.
+       (yydefault): Don't fall through to `yyresume': `goto' there.
+       * src/output.c (output_parser): When the `$' is met, skip the end
+       of its line.
+       New variable, `number_of_dollar_signs', to check there's exactly
+       one `$' in the parser skeleton.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * lib/xstrdup.c: New file, from the fileutils.
+       * src/reader.c (parse_token_decl, get_type_name, parse_type_decl)
+       (parse_assoc_decl, parse_thong_decl, get_type): Use `xstrdup'
+       instead of strlen + xmalloc + strcpy.
+       * src/symtab.c (copys): Remove, use xstrdup instead.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/gram.h (associativity): New enum type which replaces the
+       former CPP macros `RIGHT_ASSOC', `LEFT_ASSOC' and `NON_ASSOC' with
+       `right_assoc', `left_assoc' and `non_assoc'.
+       Adjust all dependencies.
+       * src/reader.c: Formatting changes.
+       (LTYPESTR): Don't define it, use it as a literal in
+       `reader_output_yylsp'.
+       * src/symtab.h (symbol_class): New enum type which replaces the
+       former CPP macros `SUNKNOWN', `STOKEN and `SNTERM' with
+       `sunknown', `stoken and `snterm'.
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/getargs.c (fixed_outfiles): Rename as...
+       (yaccflag): for consistency and accuracy.
+       Adjust dependencies.
+
+       
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       Use the more standard files `xalloc.h' and `xmalloc.c' instead of
+       Bison's `allocate.c' and `alloc.h'.  This patch was surprisingly
+       difficult and introduced a lot of core dump.  It turns out that
+       Bison used an implementation of `xmalloc' based on `calloc', and
+       at various places it does depend upon the initialization to 0.  I
+       have not tried to isolate the pertinent places, and all the former
+       calls to Bison's `xmalloc' are now using `XCALLOC'.  Someday,
+       someone should address this issue.
+
+       * src/allocate.c, src/alloc.h, m4/bison-decl.m4: Remove.
+       * lib/xmalloc.c, lib/xalloc.h, m4/malloc.m4, m4/realloc.m4: New
+       files.
+       Adjust dependencies.
+       * src/warshall.h: New file.
+       Propagate.
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       Various anti-`extern in *.c' changes.
+
+       * src/system.h: Include `assert.h'.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/state.h (nstates, final_state, first_state, first_shift)
+       (first_reduction): Move their exportation from here...
+       * src/LR0.h: to here.
+       Adjust dependencies.
+       * src/getargs.c (statisticsflag): New variable.
+       Add support for `--statistics'.
+       Adjust dependencies.
+
+       Remove a lot of now useless `extern' statements in most files.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/LR0.h: New file.
+       Propagate its use.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/print.h: New file.
+       Propagate its use.
+       * src/print.c: Formatting and ordering changes.
+       (verbose, terse): Replace with...
+       (print_results): this new function.
+       Adjust dependencies.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/conflicts.c (conflict_report): New function.
+       (conflict_log, verbose_conflict_log): Replace with...
+       (print_conflicts): this function.
+       Adjust dependencies.
+       * src/conflicts.h: New file.
+       Propagate its inclusion.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/nullable.h: New file.
+       Propagate its inclusion.
+       * src/nullable.c: Formatting changes.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/reduce.h: New file.
+       Propagate its inclusion.
+       * src/reduce.c: Topological sort and other formatting changes.
+       (bool, TRUE, FALSE): Move their definition to...
+       * src/system.h: here.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/files.c: Formatting changes.
+       (tryopen, tryclose, openfiles): Rename as...
+       (xfopen, xfclose, open_files): this.
+       (stringappend): static.
+       * src/files.h: Complete the list of exported symbols.
+       Propagate its use.
+
+
+2000-10-02  Akim Demaille  <akim@epita.fr>
+
+       * src/reader.h: New file.
+       Propagate its use instead of tedious list of `extern' and
+       prototypes.
+       * src/reader.c: Formatting changes, topological sort,
+       s/register//.
+
+
 2000-10-02  Akim Demaille  <akim@epita.fr>
 
        * src/lex.h: Prototype `lex.c' exported functions.
        (safegetc): Rename as...
        (xgetc): this.
 
-       
+
 2000-10-02  Akim Demaille  <akim@epita.fr>
 
        * src/lalr.h: New file.
        Propagate its inclusion instead of prototypes and `extern'.
        * src/lalr.c: Formatting changes, topological sorting etc.
 
-       
+
 2000-10-02  Akim Demaille  <akim@epita.fr>
 
        * src/output.c (token_actions): Introduce a temporary array,
        YYDEFACT, that makes it possible for this function to use
        output_short_table.
 
-       
+
 2000-10-02  Akim Demaille  <akim@epita.fr>
 
        `user_toknums' is output as a `short[]' in `output.c', while it is