+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/closure.c (set_fderives): De-obfuscate.
+
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/closure.c (print_firsts, print_fderives): De-obfuscate.
+
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/closure.c (set_firsts): De-obfuscate.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/output.c (action_row): De-obfuscate
+ using the good o' techniques: arrays not pointers, variable
+ locality, BITISSET, RESETBIT etc.
+
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ Pessimize the code to simplify it: from now on, all the states
+ have a valid SHIFTS, which NSHIFTS is possibly 0.
+
+ * src/LR0.c (shifts_new): Be global and move to..
+ * src/state.c, src/state.h: here.
+ * src/conflicts, src/lalr.c, src/output.c, src/print.c,
+ * src/print_graph: Adjust.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/state.h (SHIFT_DISABLE, SHIFT_IS_DISABLED): New.
+ * src/conflicts.c: Use it.
+ Restore a few missing `if (!SHIFT_IS_DISABLED)' which were
+ incorrectly ``simplified''.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (flush_shift, resolve_sr_conflict): De-obfuscate
+ using the good o' techniques: arrays not pointers, variable
+ locality, BITISSET, RESETBIT etc.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/state.h (SHIFT_SYMBOL): New.
+ * src/conflicts.c: Use it to deobfuscate.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (count_sr_conflicts, count_rr_conflicts)
+ (print_reductions): De-obfuscate using the good o' techniques:
+ arrays not pointers, variable locality, BITISSET.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (print_reductions): Arrays, not pointers.
+ Use BITISSET.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (print_reductions): Pessimize, but clarify.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (print_reductions): Improve variable locality.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (print_reductions): Pessimize, but clarify.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (print_reductions): Improve variable locality.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/state.h (SHIFT_IS_ERROR, SHIFT_IS_GOTO, SHIFT_IS_SHIFT): New.
+ * src/lalr.c: Use them.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/LR0.c (augment_automaton): Formatting changes.
+ Better variable locality.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/lalr.c (matrix_print): New.
+ (transpose): Use it.
+ Use arrays instead of pointers.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/lalr.c (maxrhs): Move to...
+ * src/gram.c, src/gram.h (ritem_longest_rhs): here.
+ * src/lalr.c (build_relations): Adjust.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/lalr.c (transpose): Free the memory allocated to the
+ argument, as it is replaced by the results by the unique caller.
+ (build_relations): Merely invoke transpose: it handles the memory
+ deallocation.
+ Improve variable locality.
+ Avoid variables used as mere abbreviations.
+ (compute_lookaheads): Use arrays instead of pointers.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/lalr.c (initialize_F): Improve variable locality.
+ Avoid variables used as mere abbreviations.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/derives.c (print_derives): Display the ruleno.
+ * src/lalr.c (initialize_F, transpose): Better variable locality
+ to improve readability.
+ Avoid variables used as mere abbreviations.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/lalr.c (traverse): Use arrays instead of pointers.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/nullable.c (set_nullable): Use a for loop to de-obfuscate
+ the handling of squeue.
+ `symbol >= 0' is wrong now, use `rule_table[ruleno].useful'.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ Because useless nonterminals are now kept alive (instead of being
+ `destroyed'), we now sometimes examine them, and store information
+ related to them. Hence we need to know their number, and adjust
+ memory allocations.
+
+ * src/reduce.c, src/reduce.h (nuseless_nonterminals): No longer
+ static.
+ * src/LR0.c (allocate_itemsets): The memory allocated to
+ `symbol_count' was used for two different purpose: once to count
+ the number of occurrences of each symbol, and later reassigned to
+ `shift_symbol', containing the symbol that can be shifted from a
+ given state.
+ Deobfuscate, i.e., allocate, use and free `symbol_count' here
+ only, and...
+ (new_itemsets): Allocate `shift_symbol' here.
+ (allocate_itemsets): symbol_count includes useless nonterminals.
+ Make room for them.
+ (free_storage): Use `free', not `XFREE', for pointers that cannot
+ be null.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/nullable.c (set_nullable): Deobfuscate the handling of
+ ritem.
+ `symbol >= 0' is wrong now, use `rule_table[ruleno].useful'.
+
+2001-12-05 Akim Demaille <akim@epita.fr>
+
+ * src/gram.c, src/gram.h (ritem_print): New.
+ * src/gram.c (dummy): Remove, now there is actual code in gram.c.
+ (This useless function was defined only to work around VMS linkers
+ that can't handle compilation units with variables only).
+ * src/reduce.c (dump_grammar): Use it to trace the construction of
+ ritem.
+
+2001-12-04 Paul Eggert <eggert@twinsun.com>
+
+ * src/bison.simple (union yyalloc): Change member names
+ to be the same as the stack names.
+ (yyparse): yyptr is now union yyalloc *, not char *.
+ (YYSTACK_RELOCATE): Likewise. This avoids a GCC warning,
+ and may generate better code on some machines.
+ (yystpcpy): Use prototype if __STDC__ is defined, not just
+ if __cplusplus is defined.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * configure.in (WARNING_CFLAGS): Add -Werror when possible.
+ (CFLAGS): Do not include the WARNING_CFLAGS here, since GNU
+ Gettext doesn't compile cleanly, and dies with -Werror.
+ * src/Makefile.am, lib/Makefile.am, tests/atlocal.in (CFLAGS):
+ Include WARNING_CFLAGS here.
+ * lib/xstrdup.c: Include xalloc.h, so that xstrdup be declared
+ before being defined.
+
+2001-11-27 Paul Eggert <eggert@twinsun.com>
+
+ * lib/quotearg.h (quotearg_n, quotearg_n_style):
+ First arg is int, not unsigned.
+ * lib/quotearg.c (quotearg_n, quotearg_n_style): Likewise.
+ (SIZE_MAX, UINT_MAX): New macros.
+ (quotearg_n_options): Abort if N is negative.
+ Avoid overflow check on hosts where size_t is 64 bits and int
+ is 32 bits, as overflow is impossible there.
+ Fix off-by-one typo that caused unnecessary reallocation.
+
+2001-11-29 Paul Eggert <eggert@twinsun.com>
+
+ Name space cleanup in generated parser.
+
+ * doc/bison.texinfo (Bison Parser): Discuss system headers
+ and their effect on the user name space.
+
+ * src/bison.simple:
+ (YYSTACK_ALLOC, YYSTACK_FREE, union yyalloc, YYSTACK_GAP_MAX,
+ YYSTACK_BYTES, YYSTACK_RELOCATE): Do not define unless necessary,
+ i.e. unless ! defined (yyoverflow) || defined (YYERROR_VERBOSE).
+
+ (YYSIZE_T): New macro. Use it instead of size_t, to avoid infringing
+ on user names when possible.
+
+ (YYSTACK_USE_ALLOCA): Do not define; just use any existing defn.
+ Simplify test for whather <alloca.h> exists.
+
+ (<stdlib.h>): Include if we will use malloc, and if standard C or C++.
+
+ (<stdio.h>): Include if YYDEBUG.
+
+ (yymemcpy): Renamed from __yy_memcpy. Do not define unless
+ ! defined (yyoverflow) && ! defined (yymemcpy).
+
+ (yymemcpy, yyparse): Rename local variables as needed so that
+ they all begin with 'yy'.
+
+ (yystrlen, yystpcpy): New functions.
+
+ (YY_DECL_NON_LSP_VARIABLES): Renamed from _YY_DECL_VARIABLES.
+ All uses changed.
+
+ (yyparse): size_t -> YYSIZE_T. Use yystrlen and yystpcpy
+ instead of relying on string.h functions. Use YYSTACK_ALLOC
+ and YYSTACK_FREE instead of malloc and free.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * src/bison.simple (YYSTYPE, YYLTYPE): Move their definitions
+ before their first uses.
+ (YYBISON, YYPURE): Move to the top of the output.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * tests/reduce.at (Useless Nonterminals): Fix.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * src/bison.simple (YYSTACK_FREE): Use `do {;} while (0)' as empty
+ if body instead of `;' to pacify GCC's warnings.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ Instead of mapping the LHS of unused rules to -1, keep the LHS
+ valid, but flag the rules as invalid.
+
+ * src/gram.h (rule_t): `useful' is a new member.
+ * src/print.c (print_grammar): Adjust.
+ * src/derives.c (set_derives): Likewise.
+ * src/reader.c (packgram, reduce_output): Likewise.
+ * src/reduce.c (reduce_grammar_tables): Likewise.
+ * tests/reduce.at (Underivable Rules, Useless Rules): New.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * src/reduce.c (reduce_output): Formatting changes.
+ * src/print.c (print_results, print_grammar): Likewise.
+ * tests/regression.at (Rule Line Numbers)
+ (Solved SR Conflicts, Unresolved SR Conflicts): Adjust.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * src/reduce.c (nonterminals_reduce): Instead of throwing away
+ useless nonterminals, move them at the end of the symbol arrays.
+ (reduce_output): Adjust.
+ * tests/reduce.at (Useless Nonterminals): Adjust.
+
+2001-11-30 Akim Demaille <akim@epita.fr>
+
+ * src/reduce.c: Various comment/formatting changes.
+ (nonterminals_reduce): New, extracted from...
+ (reduce_grammar_tables): here.
+ (reduce_grammar): Call nonterminals_reduce.
+
+2001-11-29 Paul Eggert <eggert@twinsun.com>
+
+ * src/bison.simple (YYSTACK_REALLOC): Remove.
+ (YYSTACK_ALLOC): Resurrect this macro, with its old meaning.
+ (YYSTACK_FREE, YYSTACK_GAP_MAX, YYSTACK_BYTES, YYSTACK_RELOCATE):
+ New macros.
+ (union yyalloc): New type.
+ (__yy_memcpy): Last arg is size_t, not unsigned int, to remove
+ an arbitrary restriction on hosts where size_t is wider than int.
+
+ (yyparse): Don't dump core if alloca or malloc fails; instead, report
+ a parser stack overflow. Allocate just one block of memory for all
+ three stacks, instead of allocating three blocks; this typically is
+ faster and reduces fragmentation.
+
+ Do not limit the number of items in the stack to a value that fits
+ in 'int', as this is an arbitrary limit on hosts with 64-bit
+ size_t and 32-bit int.
+
2001-11-29 Marc Autret <autret_m@epita.fr>
* tests/calc.at [AT_DATA_CALC_Y]: Use %error-verbose instead