+2002-08-02 Akim Demaille <akim@epita.fr>
+
+ * src/conflicts.c (conflicts_output): Don't output rules never
+ reduced here, since anyway that computation doesn't work.
+ * src/gram.h, src/gram.h (rule_filter_t, rule_useful_p)
+ (rule_useless_p, rule_never_reduced_p): New.
+ (grammar_rules_partial_print): Use a filter instead of a range.
+ Display the title only if needed.
+ (grammar_rules_print): Adjust.
+ (grammar_rules_never_reduced_report): New.
+ * src/tables.c (action_row): Move the computation of rules never
+ reduced to...
+ (token_actions): here.
+ * src/main.c (main): Make the parser before making the report, so
+ that rules never reduced are computed.
+ Call grammar_rules_never_reduced_report.
+ * src/print.c (print_results): Report rules never reduced.
+ * tests/conflicts.at, tests/reduce.at: Adjust.
+
+2002-08-01 Akim Demaille <akim@epita.fr>
+
+ Instead of attaching lookaheads and duplicating the rules being
+ reduced by a state, attach the lookaheads to the reductions.
+
+ * src/state.h (state_t): Remove the `lookaheads',
+ `lookaheads_rule' member.
+ (reductions_t): Add a `lookaheads' member.
+ Use a regular array for the `rules'.
+ * src/state.c (reductions_new): Initialize the lookaheads member
+ to 0.
+ (state_rule_lookaheads_print): Adjust.
+ * src/state.h, src/state.c (state_reductions_find): New.
+ * src/conflicts.c (resolve_sr_conflict, set_conflicts)
+ (count_rr_conflicts): Adjust.
+ * src/lalr.c (LArule): Remove.
+ (add_lookback_edge): Adjust.
+ (state_lookaheads_count): New.
+ (states_lookaheads_initialize): Merge into...
+ (initialize_LA): this.
+ (lalr_free): Adjust.
+ * src/main.c (main): Don't free nullable and derives too early: it
+ is used by --verbose.
+ * src/print.c, src/print_graph.c, src/tables.c: Adjust.
+
+2002-08-01 Akim Demaille <akim@epita.fr>
+
+ * src/derives.h, src/derives.c (derives): A `rule_t***' instead of
+ `rule_number_t**'.
+ (set_derives, free_derives): Rename as...
+ (derives_compute, derives_free): this.
+ Adjust all dependencies.
+ * src/nullable.c (set_nullable, free_nullable): Rename as...
+ (nullable_compute, nullable_free): these.
+ (rule_list_t): Store rule_t *, not rule_number_t.
+ * src/state.c (state_rule_lookaheads_print): Directly compare rule
+ pointers, instead of their numbers.
+ * src/main.c (main): Call nullable_free, and derives_free earlier,
+ as they were lo longer used.
+
+2002-08-01 Akim Demaille <akim@epita.fr>
+
+ * lib/timevar.c (get_time): Include children time.
+ * src/lalr.h (LA, LArule): Don't export them: used with the
+ state_t.
+ * src/lalr.c (LA, LArule): Static.
+ * src/lalr.h, src/lalr.c (lalr_free): New.
+ * src/main.c (main): Call it.
+ * src/tables.c (pack_vector): Check whether loc is >= to the
+ table_size, not >.
+ (pack_tables): Don't free froms, tos, conflict_tos, and pos...
+ (tables_generate): do it, since that's also it which allocates
+ them.
+ Don't free LA and LArule, main does.
+
+2002-07-31 Akim Demaille <akim@epita.fr>
+
+ Separate parser tables computation and output.
+
+ * src/output.c (nvectors, base_t, base, base_ninf, conflict_table)
+ (conflict_list, conflict_list_cnt, table, check, table_ninf)
+ (yydefgoto, yydefact, high): Move to...
+ * src/tables.h, src/tables.c: here.
+ * src/output.c (vector_number_t, VECTOR_NUMBER_MAX)
+ (VECTOR_NUMBER_MIN, state_number_to_vector_number)
+ (symbol_number_to_vector_number, nvectors, BASE_MAX, BASE_MIN)
+ (froms, tos, conflict_tos, tally, width, action_t, ACTION_MAX)
+ (ACTION_MIN, actrow, order, nentries, pos, conflrow)
+ (conflict_list_free, table_size, lowzero, table_grow, conflict_row)
+ (action_row, save_row, token_actions, save_column, default_goto)
+ (goto_actions, sort_actions, matching_state, pack_vector)
+ (table_ninf_remap, pack_table, prepare_actions): Move to...
+ * src/tables.c: here.
+ * src/tables.h, src/tables.c(tables_generate, tables_free): New.
+ * src/output.c (token_actions, output_base, output_conflicts)
+ (output_check): Merge into...
+ (prepare_actions): this.
+ (actions_output): Rename as...
+ (user_actions_output): this.
+ * src/main.c (main): Call tables_generate and tables_free.
+
+2002-07-31 Akim Demaille <akim@epita.fr>
+
+ Steal GCC's --time-report support.
+
+ * lib/timevar.c, lib/timevar.h, lib/timevar.def: New,
+ stolen/adjusted from GCC.
+ * m4/stage.m4: Remove time related checks.
+ * m4/timevar.m4: New.
+ * configure.in: Adjust.
+ * src/system.h: Adjust to using timevar.h.
+ * src/getargs.h, src/getargs.c: Support trace_time for
+ --trace=time.
+ * src/main.c (stage): Remove.
+ (main): Replace `stage' invocations with timevar calls.
+ * src/output.c: Insert pertinent timevar calls.
+
+2002-07-31 Akim Demaille <akim@epita.fr>
+
+ Let --trace have arguments.
+
+ * src/getargs.h (enum trace_e): New.
+ * src/getargs.c (trace_args, trace_types, trace_argmatch): New.
+ (long_options, short_options): --trace/-T takes an optional
+ argument.
+ Change all the uses of trace_flag to reflect the new flags.
+ * tests/sets.at (Firsts, Nullable, Broken Closure): Use --trace=sets.
+
+ Strengthen `stage' portability.
+
+ * m4/stage.m4 (BISON_PREREQ_STAGE): New.
+ * configure.in: Use it.
+ Don't check for malloc.h and sys/times.h.
+ * src/system.h: Include them when appropriate.
+ * src/main.c (stage): Compile only when mallinfo, struct mallinfo,
+ times and struct tms are available.
+
+2002-07-30 Akim Demaille <akim@epita.fr>
+
+ In verbose parse error message, don't report `error' as an
+ expected token.
+ * tests/actions.at (Printers and Destructors): Adjust.
+ * tests/calc.at (Calculator $1): Adjust.
+ * data/yacc.c, data/glr.c, data/lalr1.c: When making the verbose
+ error message, do not report the parser accepts the error token in
+ that state.
+
+2002-07-30 Akim Demaille <akim@epita.fr>
+
+ Normalize conflict related messages.
+
+ * src/complain.h, src/complain.c (warn, complain): New.
+ * src/conflicts.c (conflicts_print): Use them.
+ (conflict_report_yacc): New, extracted from...
+ (conflicts_print): here.
+ * tests/conflicts.at, tests/existing.at: Adjust.
+
+2002-07-30 Akim Demaille <akim@epita.fr>
+
+ Report rules which are never reduced by the parser: those hidden
+ by conflicts.
+
+ * src/LR0.c (save_reductions): Don't make the final state too
+ different: save its reduction (accept) instead of having a state
+ without any action (no shift or goto, no reduce).
+ Note: the final state is now a ``regular'' state, i.e., the
+ parsers now contain `reduce 0' as default reduction.
+ Nevertheless, since they decide to `accept' when yystate =
+ final_state, they still will not reduce rule 0.
+ * src/print.c (print_actions, print_reduction): Adjust.
+ * src/output.c (action_row): Track reduced rules.
+ (token_actions): Report rules never reduced.
+ * tests/conflicts.at, tests/regression.at: Adjust.
+
+2002-07-30 Akim Demaille <akim@epita.fr>
+
+ `stage' was accidently included in a previous patch.
+ Initiate its autoconfiscation.
+
+ * configure.in: Look for malloc.h and sys/times.h.
+ * src/main.c (stage): Adjust.
+ Report only when trace_flag.
+
+2002-07-29 Akim Demaille <akim@epita.fr>
+
+ * src/state.h, src/state.c (transitions_t): Holds state_t*'s, not
+ state_number_t.
+ (errs_t): symbol_t*, not symbol_number_t.
+ (reductions_t): rule_t*, not rule_number_t.
+ (FOR_EACH_SHIFT): New.
+ * src/LR0.c, src/conflicts.c, src/lalr.c, src/output.c
+ * src/print.c, src/print_graph.c: Adjust.
+
+2002-07-29 Akim Demaille <akim@epita.fr>
+
+ Use $accept and $end, as BYacc and BTYacc do, instead of $axiom and $.
+
+ * src/symtab.h, src/symtab.c (eoftoken, axiom): Rename as...
+ (endtoken, accept): these.
+ * src/reader.c (reader): Set endtoken's default tag to "$end".
+ Set undeftoken's tag to "$undefined" instead of "$undefined.".
+ * doc/bison.texinfo (Table of Symbols): Mention $accept and $end.
+ Adjust.
+
+2002-07-29 Akim Demaille <akim@epita.fr>
+
+ * src/reduce.c (reduce_grammar): When the language is empty,
+ complain about the start symbol, not the axiom.
+ Use its location.
+ * tests/reduce.at (Empty Language): New.
+
+2002-07-26 Akim Demaille <akim@epita.fr>
+
+ * src/reader.h, src/reader.c (gram_error): ... can't get
+ yycontrol without making too strong assumptions on the parser
+ itself.
+ * src/output.c (prepare_tokens): Use the real 0th value of
+ token_translations instead of `0'.
+ * src/parse-gram.y (yyerror): Don't rely on yycontrol being
+ visible here.
+ * data/yacc.c (yyreport_parse_error): Rename yylocation as yylloc
+ for the time being: %locations ought to provide it to yyerror.
+
+2002-07-25 Akim Demaille <akim@epita.fr>
+
+ * src/output.c (prepare_tokens): Go up to ntokens, not ntokens + 1.
+ * doc/bison.texinfo (Decl Summary): s/$illegal/$undefined./.
+ * tests/regression.at (Web2c Actions): Adjust.
+
+2002-07-25 Akim Demaille <akim@epita.fr>
+
+ Stop storing rules from 1 to nrules + 1.
+
+ * src/LR0.c, src/closure.c, src/derives.c, src/gram.c, src/lalr.c
+ * src/nullable.c, src/output.c, src/print.c, src/reader.c
+ * src/reduce.c: Allocate and free from &rules[0], not &rules[1].
+ Iterate from 0 to nrules.
+ Use rule_number_as_item_number and item_number_as_rule_number.
+ Adjust to `derive' now containing possibly 0.
+ * src/gram.h (rule_number_as_item_number, item_number_as_rule_number):
+ Handle the `- 1' part in rule numbers from/to item numbers.
+ * src/conflicts.c (log_resolution): Fix the message which reversed
+ shift and reduce.
+ * src/output.c (action_row): Initialize default_rule to -1.
+ (token_actions): Adjust.
+ * tests/sets.at (Nullable, Firsts): Fix the previously bogus
+ expected output.
+ * tests/conflicts.at (Resolved SR Conflicts): Likewise.
+
2002-07-25 Akim Demaille <akim@epita.fr>
* data/c.m4 (b4_c_function, b4_c_ansi_args, b4_c_ansi_arg)
projects / bison.git / blobdiff