+2002-12-15 Paul Eggert <eggert@twinsun.com>
+
+ Undo most of the 2002-11-12 yychar -> yytoken patch, as it broke
+ GCC 3.2.1 (which depends on yychar == YYEMPTY when there is no
+ lookahead symbol, and which sets yychar in parser actions) and it
+ disagreed with the Bison documentation. This should fix the bug
+ reported by Andrew Walrond in
+ <http://mail.gnu.org/pipermail/bug-bison/2002-December/001949.html>.
+
+ * data/yacc.c (YYTRANSLATE): Don't check for negative argument,
+ as the caller now does that.
+ (yyclearin, YYBACKUP, yyparse): Use yychar, not yytoken.
+ (YYEMPTY): Parenthesize right hand side, since others use it.
+ (yyparse): Don't assume that our generated code is the only code
+ that sets yychar.
+
+2002-12-13 Paul Eggert <eggert@twinsun.com>
+
+ Version 1.75d.
+
+ POSIX requires a "yacc" command.
+ * src/Makefile.am (bin_SCRIPTS): New macro, for yacc.
+ (MOSTLYCLEANFILES): Add yacc.
+ (yacc): New rule.
+ * doc/bison.texinfo (Invocation, Bison Options): Mention yacc
+ as an alias for bison y.
+
+ * po/LINGUAS: Add da.
+
+ * src/getargs.c (__GNU_LIBRARY__) [lint]: Define to work around
+ problem with latest <getopt.h>.
+ (HACK_FOR___GNU_LIBRARY___PROTOTYPE): New macro.
+
+ * doc/fdl.texi: Upgrade to 1.2.
+ * lib/alloca.c, lib/error.c, lib/getopt.c, lib/getopt.h
+ lib/getopt1.c, lib/gettext.h, lib/memrchr.c, lib/obstack.c,
+ lib/obstack.h, lib/strnlen.c, lib/unlocked-io.h: Sync with
+ gnulib.
+ * config/install-sh: Sync with autotools.
+
+ Fix a bad interaction with flex 2.5.23 reported by Bruce Lilly in
+ <http://mail.gnu.org/pipermail/bug-bison/2002-November/001929.html>.
+ * data/yacc.c (YYLTYPE, struct yyltype): Do not define unless
+ locations are requested.
+ (union yyalloc): Do not depend on YYLTYPE_IS_TRIVIAL unless
+ locations are requested.
+
+2002-12-12 Paul Eggert <eggert@twinsun.com>
+
+ Remove unportable casts and storage allocation tricks.
+ While we're at it, remove almost all casts, since they
+ usually aren't needed and are a sign of trouble.
+
+ * configure.ac (AC_CHECK_TYPES): Check for uintptr_t.
+
+ * src/derives.c (derives_compute): Do not subtract NTOKENS from
+ the pointer DSET returned by malloc; this isn't portable.
+ Instead, always use DSET[i - NTOKENS] rather than DSET[i].
+ Similarly for DERIVES.
+ * src/lalr.c (set_goto_map): Likewise, for GOTO_MAP and TEMP_MAP.
+ * src/nullable.c (nullable_compute): Likewise, for RSETS and NULLABLE.
+ * src/reduce.c (reduce_grammar_tables): Likewise, for nontermmap
+
+ * src/derives.c (derives_compute): Do not bother invoking
+ int_of_rule_number, since rule numbers are integers.
+
+ * src/files.c (concat2, tr, compute_base_name): Use xmalloc (N)
+ rather than XMALLOC (char, N).
+
+ * src/files.c (filename_split): Rewrite to avoid cast.
+
+ * src/gram.h (symbol_number_as_item_number,
+ item_number_as_symbol_number, rule_number_as_item_number,
+ item_number_as_rule_number):
+ Now inline functions rather than macros, to avoid casts.
+ * src/state.h (state_number_as_int): Likewise.
+ * src/tables.c (state_number_to_vector_number,
+ symbol_number_to_vector_number): Likewise.
+
+ * src/gram.h (int_of_rule_number): Remove; no longer used.
+
+ * src/lalr.c (add_lookback_edge): Use malloc rather than calloc,
+ since the resulting storage is always stored into.
+
+ * src/main.c (alloca) [C_ALLOCA]: Add decl here, the only place
+ where it's needed.
+
+ * src/muscle_tab.c (muscle_m4_output):
+ Now inline. Return bool, not int.
+ * src/state.c (state_compare): Likewise.
+ * src/symtab.c (symbol_check_defined,
+ symbol_check_alias_consistency, symbol_pack, symbol_translation,
+ hash_compare_symbol, hash_symbol):
+ Likewise.
+ * src/uniqstr.c (uniqstr_print): Likewise.
+ * src/muscle_tab.c (muscle_m4_output_processor):
+ New function, to avoid casts.
+ * src/state.c (state_comparator, stage_hasher): Likewise.
+ * src/symtab.c (symbol_check_defined_processor,
+ symbol_check_alias_consistency_processor, symbol_pack_processor,
+ symbol_translation_processor, hash_symbol_comparator,
+ hash_symbol_hasher): Likewise.
+ * src/uniqstr.c (uniqstr_print_processor): Likewise.
+ * src/muscle_tab.c (muscles_m4_output):
+ Use new functions instead of casting old functions unportably.
+ * src/state.c (state_hash_new): Likewise.
+ * src/symtab.c (symbols_new, symbols_do, symbols_check_defined,
+ symbols_token_translations_init):
+ Likewise.
+ * src/uniqstr.c (uniqstrs_new, hash_initialize, uniqstrs_do): Likewise.
+
+ * src/output.c (GENERATE_MUSCLE_INSERT_TABLE): Use long local
+ var instead of casting to long, to avoid casts.
+ (prepare_states): Use MALLOC rather than alloca, so that we don't
+ have to worry about alloca.
+ * src/state.c (state_hash_lookup): Likewise.
+
+ * src/scan-gram.l (<SC_ESCAPED_CHARACTER>"'"): Use unsigned char
+ local var instead of casting to unsigned char, to avoid casts.
+
+ * src/state.c (TRANSITIONS_ALLOC, ERRS_ALLOC, REDUCTIONS_ALLOC,
+ STATE_ALLOC): Remove.
+ (transitions_new, errs_new, reductions_new, state_new): Use malloc
+ rather than calloc, and use offsetof to avoid allocating slightly
+ too much storage.
+ (state_new): Initialize all members.
+
+ * src/state.c (state_hash): Use unsigned accumulator, not signed.
+
+ * src/symtab.c (symbol_free): Remove; unused.
+ (symbol_get): Remove cast in lhs of assignment.
+ (symbols_do): Now static. Accept generic arguments, not
+ hashing-related ones.
+
+ * src/symtab.h: (NUMBER_UNDEFINED): Remove unnecessary cast.
+ (symbol_processor): Remove.
+ (symbols_do): Remove decl; now static.
+
+ * src/system.h (alloca): Remove; decl no longer needed.
+ (<stddef.h>): Include, for offsetof.
+ (<inttypes.>, <stdint.h>): Include if available.
+ (uintptr_t): New type, if system lacks it.
+ (CALLOC, MALLOC, REALLOC): New macros.
+ All uses of XCALLOC, XMALLOC, and XREALLOC changed to use these
+ new macros.
+
+ * src/tables.c (table_size): Now int, to pacify GCC.
+ (table_grow, table_ninf_remap): Use signed table size.
+ (save_row): Don't bother initializing locals when not needed.
+ (default_goto, goto_actions, pack_vector): Remove unnecessary casts.
+ * src/uniqstr.c (hash_compare_uniqstr): Likewise.
+
+ * src/vcg.h: Correct misspellings.
+
+ * src/vcg_defaults.h (G_CMAX): Now INT_MAX.
+
+
+ * src/getargs.c (getargs): Don't assume EOF == -1.
+
2002-12-09 Paul Eggert <eggert@twinsun.com>
Change identifier spellings to avoid collisions with names
projects / bison.git / blobdiff