projects / bison.git / blame
| Commit | Line | Data |
|---|---|---|
| 0ab3728b AD |
1 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 2 | ||
| 3 | * src/LR0.c (insert_start_shift): Rename as... | |
| 4 | (insert_start_shifting_state): this. | |
| 5 | (insert_eof_shifting_state, insert_accepting_state): New. | |
| 6 | (augment_automaton): Adjust. | |
| 7 | Better locality of the variables. | |
| 8 | When looking if the start_symbol is shifted from the initial | |
| 9 | state, using `while (... symbol != start_symbol ...)' sounds | |
| 10 | better than `while (... symbol < start_symbol ...)': If fail | |
| 11 | to see how the order between symbols could be relevant! | |
| 12 | ||
| 78af9bbc AD |
13 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 14 | ||
| 15 | * src/getargs.h: Don't declare `spec_name_prefix' and | |
| 16 | `spec_file_prefix', declared by src/files.h. | |
| 17 | * src/files.c, src/files.h: Default for spec_name_prefix is "yy". | |
| 18 | * src/muscle_tab.c (muscle_init): Default prefix to NULL. | |
| 19 | * src/output.c (prepare): Adjust. | |
| 20 | * src/reader.c (symbols_output): Likewise. | |
| 21 | * src/vmsgetargs.c: Vaguely adjust, but who cares? | |
| 22 | ||
| 23 | ||
| bdef2a41 AD |
24 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 25 | ||
| 26 | * src/muscle_tab.c (muscle_init): NULL is a better default than | |
| 27 | `"0"'. | |
| 28 | ||
| 29 | ||
| 3735969c AD |
30 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 31 | ||
| 32 | * src/reader.c (reader): Calling symbols_output once is enough. | |
| 33 | ||
| 34 | ||
| 49701457 AD |
35 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 36 | ||
| 37 | Now that states have a complete set of members, the linked list of | |
| 38 | reductions is useless: just fill directly the state's reductions | |
| 39 | member. | |
| 40 | ||
| 41 | * src/state.h (struct reductions): Remove member `number' and | |
| 42 | `next'. | |
| 43 | * src/LR0.c (first_reduction, last_reduction): Remove. | |
| 44 | (save_reductions): Don't link the new reductions, store them in | |
| 45 | this_state. | |
| 46 | * src/lalr.c (set_state_table): No need to attach reductions to | |
| 47 | states, it's already done. | |
| 48 | * src/output.c (output_actions): No longer free the shifts, then | |
| 49 | the reductions, then the states: free all the states and their | |
| 50 | members. | |
| 51 | ||
| 52 | ||
| 0edad749 AD |
53 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 54 | ||
| 55 | * src/options.c (OPTN, DRTV, BOTH): New. | |
| 56 | (option_table): Use them. | |
| 57 | ||
| 58 | ||
| 59 | * src/muscle_tab.c: Don't include xalloc.h and string.h: that's | |
| 60 | the job of system.h. | |
| 61 | * src/options.c: Don't include stdio.h and xalloc.h for the same | |
| 62 | reasons. | |
| 63 | ||
| 64 | ||
| 5449dd0f AD |
65 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 66 | ||
| 67 | * src/output.c (output, prepare): Make sure the values of the | |
| 68 | muscles `action' and `prologue' are 0-terminated. | |
| 69 | ||
| a870c567 AD |
70 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 71 | ||
| 72 | Clean up GCC warnings. | |
| 73 | ||
| 74 | * src/reader.c (copy_action): `buf' is not used. | |
| 75 | (parse_skel_decl): Be static. | |
| 76 | * src/muscle_tab.c (mhash1, mhash2, muscle_insert): Preserve `const'. | |
| 77 | * src/options.h (create_long_option_table): Have a real prototype. | |
| 78 | * lib/hash.c, lib/hash.h (hash_insert, hash_insert_at, hash_delete) | |
| 79 | (hash_delete_at): Return const void *. | |
| 80 | Adjust casts to preserve the const. | |
| 81 | ||
| 80df8768 AD |
82 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 83 | ||
| 84 | * configure.in: Require 2.52g. | |
| 85 | M4 is not needed, but AUTOM4TE is. | |
| 86 | * m4/m4.m4: Remove. | |
| 87 | * tests/Makefile.am: Adjust. | |
| 88 | ||
| 89 | ||
| f693ad14 AD |
90 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 91 | ||
| 92 | One structure for states is enough, even though theoretically | |
| 93 | there are LR(0) states and LALR(1) states. | |
| 94 | ||
| 95 | * src/lalr.h (state_t): Remove. | |
| 96 | (state_table): Be state_t **, not state_t *. | |
| 97 | * src/state.h (core, CORE_ALLOC): Rename as... | |
| 98 | (state_t, STATE_ALLOC): this. | |
| 99 | Add the LALR(1) members: shifts, reductions, errs. | |
| 100 | * src/LR0.c (state_table): Rename as... | |
| 101 | (state_hash): this, to avoid name clashes with the global | |
| 102 | `state_table'. | |
| 103 | * src/print_graph.c, src/LR0.c, src/LR0.h, src/conflicts.c | |
| 104 | * src/lalr.c, src/lalr.h, src/output.c, src/print.c: Adjust. | |
| 105 | ||
| 106 | ||
| 74ffbcb6 AD |
107 | 2001-12-10 Akim Demaille <akim@epita.fr> |
| 108 | ||
| 109 | Bison dumps core on bash.y. | |
| 110 | Reported by Pascal Bart. | |
| 111 | ||
| 112 | * src/warshall.c (bitmatrix_print): New. | |
| 113 | (TC): Use it. | |
| 114 | When performing a transitive closure R(i, j) && R(j, k) => R(i, k), | |
| 115 | j must be the outer loop. | |
| 116 | * tests/regression.at (Broken Closure): New. | |
| 117 | ||
| 07708e19 AD |
118 | 2001-12-05 Akim Demaille <akim@epita.fr> |
| 119 | ||
| 120 | * tests/atlocal.in (CPPFLAGS): Do not leave a space between -I and | |
| 121 | its argument. | |
| 122 |