]>
Commit | Line | Data |
---|---|---|
dbfb6dcd AD |
1 | 2001-12-12 Akim Demaille <akim@epita.fr> |
2 | ||
3 | Some hosts don't like `/' in includes. | |
4 | ||
5 | * src/system.h: Include libgettext.h without qualifying the path. | |
6 | * src/Makefile.am (INCLUDES): Add $(top_srcdir)/intl, remove | |
7 | $(top_srcdir). | |
8 | ||
c25fb648 MA |
9 | 2001-12-11 Marc Autret <autret_m@epita.fr> |
10 | ||
11 | * src/output.c (output_parser): Remove useless muscle. | |
12 | ||
710ddc4f MA |
13 | 2001-12-11 Marc Autret <autret_m@epita.fr> |
14 | ||
15 | * src/bison.simple: Remove #line just before %%epilogue. It | |
16 | is now handled in ... | |
17 | * src/reader.c (read_additionnal_code): Add the output of a | |
18 | #line for the epilogue. | |
19 | ||
e83d80b8 MA |
20 | 2001-12-10 Marc Autret <autret_m@epita.fr> |
21 | ||
22 | * src/reader.c (copy_definition): Re-use CPP-outed code which | |
23 | replace precedent remove. | |
24 | * src/bison.simple: Remove #line before %%prologue because | |
25 | %%input-line is wrong at this time. | |
26 | ||
971d5158 MA |
27 | 2001-12-10 Marc Autret <autret_m@epita.fr> |
28 | ||
29 | * src/reader.c (symbols_output): Clean up. | |
30 | * src/output.c (output_gram, output): Clean up. | |
31 | ||
5edafffd AD |
32 | 2001-12-10 Akim Demaille <akim@epita.fr> |
33 | ||
34 | * src/lalr.c (initialize_lookaheads): New. Extracted from... | |
35 | * src/LR0.c (set_state_table): here. | |
36 | * src/lalr.c (lalr): Call it. | |
37 | ||
0279f8e9 AD |
38 | 2001-12-10 Akim Demaille <akim@epita.fr> |
39 | ||
40 | * src/state.h (shifts): Remove the `number' member: shifts are | |
41 | attached to state, hence no longer need to be labelled with a | |
42 | state number. | |
43 | ||
190c4f5f AD |
44 | 2001-12-10 Akim Demaille <akim@epita.fr> |
45 | ||
46 | Now that states have a complete set of members, the linked list of | |
47 | shifts is useless: just fill directly the state's shifts member. | |
48 | ||
49 | * src/state.h (shifts): Remove the `next' member. | |
50 | * src/LR0.c (first_state, last_state): Remove. | |
51 | Adjust the callers. | |
52 | (augment_automaton): Don't look for the shifts that must be added | |
53 | a shift on EOF: it is those of the state we looked for! But now, | |
54 | since shifts are attached, it is no longer needed to looking | |
55 | merely by its id: its number. | |
56 | ||
2a73b93d AD |
57 | 2001-12-10 Akim Demaille <akim@epita.fr> |
58 | ||
59 | * src/LR0.c (augment_automaton): Better variable locality. | |
60 | Remove an impossible branch: if there is a state corresponding to | |
61 | the start symbol being shifted, then there is shift for the start | |
62 | symbol from the initial state. | |
63 | ||
74392f6a AD |
64 | 2001-12-10 Akim Demaille <akim@epita.fr> |
65 | ||
66 | * src/LR0.c (augment_automaton): Call `insert_eof_shifting_state' | |
67 | only when appropriate: when insert_start_shifting_state' is not | |
68 | invoked. | |
69 | * tests/regression.at (Rule Line Numbers): Adjust. | |
70 | ||
37c82725 AD |
71 | 2001-12-10 Akim Demaille <akim@epita.fr> |
72 | ||
73 | * src/LR0.c (augment_automaton): Now that all states have shifts, | |
74 | merge the two cases addition shifts to the initial state. | |
75 | ||
6a164e0c AD |
76 | 2001-12-10 Akim Demaille <akim@epita.fr> |
77 | ||
78 | * src/lalr.c (set_state_table): Move to... | |
79 | * src/LR0.c: here. | |
80 | * src/lalr.c (lalr): Don't call it... | |
81 | * src/LR0.c (generate_states): do it. | |
82 | * src/LR0.h (first_state): Remove, only the table is used. | |
83 | ||
7215de24 AD |
84 | 2001-12-10 Akim Demaille <akim@epita.fr> |
85 | ||
86 | * src/LR0.h (first_shift, first_reduction): Remove. | |
87 | * src/lalr.c: Don't use first_shift: find shifts through the | |
88 | states. | |
89 | ||
80e25d4d AD |
90 | 2001-12-10 Akim Demaille <akim@epita.fr> |
91 | ||
92 | * src/LR0.c: Attach shifts to states as soon as they are | |
93 | computed. | |
94 | * src/lalr.c (set_state_table): Instead of assigning shifts to | |
95 | state, just assert that the mapping was properly done. | |
96 | ||
0ab3728b AD |
97 | 2001-12-10 Akim Demaille <akim@epita.fr> |
98 | ||
99 | * src/LR0.c (insert_start_shift): Rename as... | |
100 | (insert_start_shifting_state): this. | |
101 | (insert_eof_shifting_state, insert_accepting_state): New. | |
102 | (augment_automaton): Adjust. | |
103 | Better locality of the variables. | |
104 | When looking if the start_symbol is shifted from the initial | |
105 | state, using `while (... symbol != start_symbol ...)' sounds | |
106 | better than `while (... symbol < start_symbol ...)': If fail | |
107 | to see how the order between symbols could be relevant! | |
108 | ||
78af9bbc AD |
109 | 2001-12-10 Akim Demaille <akim@epita.fr> |
110 | ||
111 | * src/getargs.h: Don't declare `spec_name_prefix' and | |
112 | `spec_file_prefix', declared by src/files.h. | |
113 | * src/files.c, src/files.h: Default for spec_name_prefix is "yy". | |
114 | * src/muscle_tab.c (muscle_init): Default prefix to NULL. | |
115 | * src/output.c (prepare): Adjust. | |
116 | * src/reader.c (symbols_output): Likewise. | |
117 | * src/vmsgetargs.c: Vaguely adjust, but who cares? | |
118 | ||
bdef2a41 AD |
119 | 2001-12-10 Akim Demaille <akim@epita.fr> |
120 | ||
121 | * src/muscle_tab.c (muscle_init): NULL is a better default than | |
122 | `"0"'. | |
123 | ||
3735969c AD |
124 | 2001-12-10 Akim Demaille <akim@epita.fr> |
125 | ||
126 | * src/reader.c (reader): Calling symbols_output once is enough. | |
127 | ||
49701457 AD |
128 | 2001-12-10 Akim Demaille <akim@epita.fr> |
129 | ||
130 | Now that states have a complete set of members, the linked list of | |
131 | reductions is useless: just fill directly the state's reductions | |
132 | member. | |
133 | ||
134 | * src/state.h (struct reductions): Remove member `number' and | |
135 | `next'. | |
136 | * src/LR0.c (first_reduction, last_reduction): Remove. | |
137 | (save_reductions): Don't link the new reductions, store them in | |
138 | this_state. | |
139 | * src/lalr.c (set_state_table): No need to attach reductions to | |
140 | states, it's already done. | |
141 | * src/output.c (output_actions): No longer free the shifts, then | |
142 | the reductions, then the states: free all the states and their | |
143 | members. | |
144 | ||
0edad749 AD |
145 | 2001-12-10 Akim Demaille <akim@epita.fr> |
146 | ||
147 | * src/options.c (OPTN, DRTV, BOTH): New. | |
148 | (option_table): Use them. | |
149 | ||
0edad749 AD |
150 | * src/muscle_tab.c: Don't include xalloc.h and string.h: that's |
151 | the job of system.h. | |
152 | * src/options.c: Don't include stdio.h and xalloc.h for the same | |
153 | reasons. | |
154 | ||
5449dd0f AD |
155 | 2001-12-10 Akim Demaille <akim@epita.fr> |
156 | ||
157 | * src/output.c (output, prepare): Make sure the values of the | |
158 | muscles `action' and `prologue' are 0-terminated. | |
159 | ||
a870c567 AD |
160 | 2001-12-10 Akim Demaille <akim@epita.fr> |
161 | ||
162 | Clean up GCC warnings. | |
163 | ||
164 | * src/reader.c (copy_action): `buf' is not used. | |
165 | (parse_skel_decl): Be static. | |
166 | * src/muscle_tab.c (mhash1, mhash2, muscle_insert): Preserve `const'. | |
167 | * src/options.h (create_long_option_table): Have a real prototype. | |
168 | * lib/hash.c, lib/hash.h (hash_insert, hash_insert_at, hash_delete) | |
169 | (hash_delete_at): Return const void *. | |
170 | Adjust casts to preserve the const. | |
171 | ||
80df8768 AD |
172 | 2001-12-10 Akim Demaille <akim@epita.fr> |
173 | ||
174 | * configure.in: Require 2.52g. | |
175 | M4 is not needed, but AUTOM4TE is. | |
176 | * m4/m4.m4: Remove. | |
177 | * tests/Makefile.am: Adjust. | |
178 | ||
f693ad14 AD |
179 | 2001-12-10 Akim Demaille <akim@epita.fr> |
180 | ||
181 | One structure for states is enough, even though theoretically | |
182 | there are LR(0) states and LALR(1) states. | |
183 | ||
184 | * src/lalr.h (state_t): Remove. | |
185 | (state_table): Be state_t **, not state_t *. | |
186 | * src/state.h (core, CORE_ALLOC): Rename as... | |
187 | (state_t, STATE_ALLOC): this. | |
188 | Add the LALR(1) members: shifts, reductions, errs. | |
189 | * src/LR0.c (state_table): Rename as... | |
190 | (state_hash): this, to avoid name clashes with the global | |
191 | `state_table'. | |
192 | * src/print_graph.c, src/LR0.c, src/LR0.h, src/conflicts.c | |
193 | * src/lalr.c, src/lalr.h, src/output.c, src/print.c: Adjust. | |
194 | ||
74ffbcb6 AD |
195 | 2001-12-10 Akim Demaille <akim@epita.fr> |
196 | ||
197 | Bison dumps core on bash.y. | |
198 | Reported by Pascal Bart. | |
199 | ||
200 | * src/warshall.c (bitmatrix_print): New. | |
201 | (TC): Use it. | |
202 | When performing a transitive closure R(i, j) && R(j, k) => R(i, k), | |
203 | j must be the outer loop. | |
204 | * tests/regression.at (Broken Closure): New. | |
205 | ||
07708e19 AD |
206 | 2001-12-05 Akim Demaille <akim@epita.fr> |
207 | ||
208 | * tests/atlocal.in (CPPFLAGS): Do not leave a space between -I and | |
209 | its argument. | |
210 |