]>
Commit | Line | Data |
---|---|---|
416bd7a9 MA |
1 | -*- outline -*- |
2 | ||
6cbfbcc5 AD |
3 | * Several %unions |
4 | I think this is a pleasant (but useless currently) feature, but in the | |
5 | future, I want a means to %include other bits of grammars, and _then_ | |
6 | it will be important for the various bits to define their needs in | |
7 | %union. | |
76551463 | 8 | |
5c0a0514 AD |
9 | When implementing multiple-%union support, bare the following in mind: |
10 | ||
11 | - when --yacc, this must be flagged as an error. Don't make it fatal | |
12 | though. | |
13 | ||
14 | - The #line must now appear *inside* the definition of yystype. | |
15 | Something like | |
16 | ||
17 | { | |
18 | #line 12 "foo.y" | |
19 | int ival; | |
20 | #line 23 "foo.y" | |
21 | char *sval; | |
22 | } | |
23 | ||
d4e7d3a1 AD |
24 | * Experimental report features |
25 | Decide whether they should be enabled, or optional. For instance, on: | |
26 | ||
27 | input: | |
28 | exp | |
29 | | input exp | |
30 | ; | |
31 | ||
32 | exp: | |
33 | token1 "1" | |
34 | | token2 "2" | |
35 | | token3 "3" | |
36 | ; | |
37 | ||
38 | token1: token; | |
39 | token2: token; | |
40 | token3: token; | |
41 | ||
42 | the traditional Bison reports: | |
43 | ||
44 | state 0 | |
45 | ||
46 | $axiom -> . input $ (rule 0) | |
47 | ||
48 | token shift, and go to state 1 | |
49 | ||
50 | input go to state 2 | |
51 | exp go to state 3 | |
52 | token1 go to state 4 | |
53 | token2 go to state 5 | |
54 | token3 go to state 6 | |
55 | ||
56 | state 1 | |
57 | ||
58 | token1 -> token . (rule 6) | |
59 | token2 -> token . (rule 7) | |
60 | token3 -> token . (rule 8) | |
61 | ||
62 | "2" reduce using rule 7 (token2) | |
63 | "3" reduce using rule 8 (token3) | |
64 | $default reduce using rule 6 (token1) | |
65 | ||
66 | while with --trace, i.e., when enabling both the display of non-core | |
67 | item sets and the display of lookaheads, Bison now displays: | |
68 | ||
69 | state 0 | |
70 | ||
71 | $axiom -> . input $ (rule 0) | |
72 | input -> . exp (rule 1) | |
73 | input -> . input exp (rule 2) | |
74 | exp -> . token1 "1" (rule 3) | |
75 | exp -> . token2 "2" (rule 4) | |
76 | exp -> . token3 "3" (rule 5) | |
77 | token1 -> . token (rule 6) | |
78 | token2 -> . token (rule 7) | |
79 | token3 -> . token (rule 8) | |
80 | ||
81 | token shift, and go to state 1 | |
82 | ||
83 | input go to state 2 | |
84 | exp go to state 3 | |
85 | token1 go to state 4 | |
86 | token2 go to state 5 | |
87 | token3 go to state 6 | |
88 | ||
89 | state 1 | |
90 | ||
91 | token1 -> token . ["1"] (rule 6) | |
92 | token2 -> token . ["2"] (rule 7) | |
93 | token3 -> token . ["3"] (rule 8) | |
94 | ||
95 | "2" reduce using rule 7 (token2) | |
96 | "3" reduce using rule 8 (token3) | |
97 | $default reduce using rule 6 (token1) | |
98 | ||
99 | so decide whether this should be an option, or always enabled. I'm in | |
100 | favor of making it the default, but maybe we should tune the output to | |
101 | distinguish core item sets from non core: | |
102 | ||
103 | state 0 | |
104 | Core: | |
105 | $axiom -> . input $ (rule 0) | |
106 | ||
107 | Derived: | |
108 | input -> . exp (rule 1) | |
109 | input -> . input exp (rule 2) | |
110 | exp -> . token1 "1" (rule 3) | |
111 | exp -> . token2 "2" (rule 4) | |
112 | exp -> . token3 "3" (rule 5) | |
113 | token1 -> . token (rule 6) | |
114 | token2 -> . token (rule 7) | |
115 | token3 -> . token (rule 8) | |
116 | ||
117 | token shift, and go to state 1 | |
118 | ||
119 | input go to state 2 | |
120 | exp go to state 3 | |
121 | token1 go to state 4 | |
122 | token2 go to state 5 | |
123 | token3 go to state 6 | |
124 | ||
125 | ||
308a2f76 AD |
126 | > So, it seems clear that it has to be an additional option :) |
127 | ||
128 | Paul: | |
129 | ||
130 | There will be further such options in the future, so I'd make | |
131 | them all operands of the --report option. E.g., you could do | |
132 | something like this: | |
133 | ||
134 | --report=state --report=lookahead --report=itemset | |
135 | --report=conflict-path | |
136 | ||
137 | where "--verbose" is equivalent to "--report=state", and where | |
138 | "--report=conflict-path" reports each path to a conflict | |
139 | state. | |
140 | ||
141 | (As a minor point, I prefer avoiding plurals in option names. | |
142 | It's partly for brevity, and partly to avoid wearing out the | |
143 | 's' keys in our keyboards. :-) | |
144 | ||
145 | To implement this, see in the Fileutils the latest versions of | |
146 | argmatch and so forth. | |
147 | ||
d4e7d3a1 | 148 | |
eaff5ee3 | 149 | * Coding system independence |
4358321a | 150 | Paul notes: |
eaff5ee3 AD |
151 | |
152 | Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is | |
153 | 255). It also assumes that the 8-bit character encoding is | |
154 | the same for the invocation of 'bison' as it is for the | |
155 | invocation of 'cc', but this is not necessarily true when | |
156 | people run bison on an ASCII host and then use cc on an EBCDIC | |
157 | host. I don't think these topics are worth our time | |
158 | addressing (unless we find a gung-ho volunteer for EBCDIC or | |
159 | PDP-10 ports :-) but they should probably be documented | |
160 | somewhere. | |
161 | ||
8b3ba7ff AD |
162 | * Output directory |
163 | Akim: | |
164 | ||
165 | | I consider this to be a bug in bison: | |
166 | | | |
167 | | /tmp % mkdir src | |
168 | | /tmp % cp ~/src/bison/tests/calc.y src | |
169 | | /tmp % mkdir build && cd build | |
170 | | /tmp/build % bison ../src/calc.y | |
171 | | /tmp/build % cd .. | |
172 | | /tmp % ls -l build src | |
173 | | build: | |
174 | | total 0 | |
175 | | | |
176 | | src: | |
177 | | total 32 | |
178 | | -rw-r--r-- 1 akim lrde 27553 oct 2 16:31 calc.tab.c | |
179 | | -rw-r--r-- 1 akim lrde 3335 oct 2 16:31 calc.y | |
180 | | | |
181 | | | |
182 | | Would it be safe to change this behavior to something more reasonable? | |
183 | | Do you think some people depend upon this? | |
184 | ||
185 | Jim: | |
186 | ||
187 | Is it that behavior documented? | |
188 | If so, then it's probably not reasonable to change it. | |
189 | I've Cc'd the automake list, because some of automake's | |
190 | rules use bison through $(YACC) -- though I'll bet they | |
191 | all use it in yacc-compatible mode. | |
192 | ||
193 | Pavel: | |
194 | ||
195 | Hello, Jim and others! | |
196 | ||
197 | > Is it that behavior documented? | |
198 | > If so, then it's probably not reasonable to change it. | |
199 | > I've Cc'd the automake list, because some of automake's | |
200 | > rules use bison through $(YACC) -- though I'll bet they | |
201 | > all use it in yacc-compatible mode. | |
202 | ||
203 | Yes, Automake currently used bison in Automake-compatible mode, but it | |
204 | would be fair for Automake to switch to the native mode as long as the | |
205 | processed files are distributed and "missing" emulates bison. | |
206 | ||
207 | In any case, the makefiles should specify the output file explicitly | |
208 | instead of relying on weird defaults. | |
209 | ||
210 | > | src: | |
211 | > | total 32 | |
212 | > | -rw-r--r-- 1 akim lrde 27553 oct 2 16:31 calc.tab.c | |
213 | > | -rw-r--r-- 1 akim lrde 3335 oct 2 16:31 calc.y | |
214 | ||
215 | This is not _that_ ugly as it seems - with Automake you want to put | |
216 | sources where they belong - to the source directory. | |
217 | ||
218 | > | This is not _that_ ugly as it seems - with Automake you want to put | |
219 | > | sources where they belong - to the source directory. | |
220 | > | |
221 | > The difference source/build you are referring to is based on Automake | |
222 | > concepts. They have no sense at all for tools such as bison or gcc | |
223 | > etc. They have input and output. I do not want them to try to grasp | |
224 | > source/build. I want them to behave uniformly: output *here*. | |
225 | ||
226 | I realize that. | |
227 | ||
228 | It's unfortunate that the native mode of Bison behaves in a less uniform | |
229 | way than the yacc mode. I agree with your point. Bison maintainters may | |
230 | want to fix it along with the documentation. | |
231 | ||
232 | ||
fa770c86 AD |
233 | * Unit rules |
234 | Maybe we could expand unit rules, i.e., transform | |
235 | ||
236 | exp: arith | bool; | |
237 | arith: exp '+' exp; | |
238 | bool: exp '&' exp; | |
239 | ||
240 | into | |
241 | ||
242 | exp: exp '+' exp | exp '&' exp; | |
243 | ||
244 | when there are no actions. This can significantly speed up some | |
245 | grammars. | |
246 | ||
51dec47b AD |
247 | * Stupid error messages |
248 | An example shows it easily: | |
249 | ||
250 | src/bison/tests % ./testsuite -k calc,location,error-verbose -l | |
251 | GNU Bison 1.49a test suite test groups: | |
252 | ||
253 | NUM: FILENAME:LINE TEST-GROUP-NAME | |
254 | KEYWORDS | |
255 | ||
256 | 51: calc.at:440 Calculator --locations --yyerror-verbose | |
257 | 52: calc.at:442 Calculator --defines --locations --name-prefix=calc --verbose --yacc --yyerror-verbose | |
258 | 54: calc.at:445 Calculator --debug --defines --locations --name-prefix=calc --verbose --yacc --yyerror-verbose | |
259 | src/bison/tests % ./testsuite 51 -d | |
260 | ## --------------------------- ## | |
261 | ## GNU Bison 1.49a test suite. ## | |
262 | ## --------------------------- ## | |
263 | 51: calc.at:440 ok | |
264 | ## ---------------------------- ## | |
265 | ## All 1 tests were successful. ## | |
266 | ## ---------------------------- ## | |
267 | src/bison/tests % cd ./testsuite.dir/51 | |
268 | tests/testsuite.dir/51 % echo "()" | ./calc | |
269 | 1.2-1.3: parse error, unexpected ')', expecting error or "number" or '-' or '(' | |
fa770c86 | 270 | |
01c56de4 AD |
271 | * yyerror, yyprint interface |
272 | It should be improved, in particular when using Bison features such as | |
273 | locations, and YYPARSE_PARAMS. For the time being, it is recommended | |
274 | to #define yyerror and yyprint to steal internal variables... | |
275 | ||
fa770c86 AD |
276 | * read_pipe.c |
277 | This is not portable to DOS for instance. Implement a more portable | |
278 | scheme. Sources of inspiration include GNU diff, and Free Recode. | |
279 | ||
aef1ffd5 AD |
280 | * Memory leaks in the generator |
281 | A round of memory leak clean ups would be most welcome. Dmalloc, | |
282 | Checker GCC, Electric Fence, or Valgrind: you chose your tool. | |
283 | ||
284 | * Memory leaks in the parser | |
285 | The same applies to the generated parsers. In particular, this is | |
286 | critical for user data: when aborting a parsing, when handling the | |
287 | error token etc., we often throw away yylval without giving a chance | |
288 | of cleaning it up to the user. | |
289 | ||
bcb05e75 MA |
290 | * --graph |
291 | Show reductions. [] | |
292 | ||
704a47c4 | 293 | * Broken options ? |
c3995d99 | 294 | ** %no-lines [ok] |
04a76783 | 295 | ** %no-parser [] |
fbbf9b3b | 296 | ** %pure-parser [] |
04a76783 MA |
297 | ** %token-table [] |
298 | ** Options which could use parse_dquoted_param (). | |
299 | Maybe transfered in lex.c. | |
300 | *** %skeleton [ok] | |
301 | *** %output [] | |
302 | *** %file-prefix [] | |
303 | *** %name-prefix [] | |
ec93a213 | 304 | |
fbbf9b3b | 305 | ** Skeleton strategy. [] |
c3a8cbaa MA |
306 | Must we keep %no-parser? |
307 | %token-table? | |
fbbf9b3b | 308 | *** New skeletons. [] |
416bd7a9 | 309 | |
c111e171 | 310 | * src/print_graph.c |
31b53af2 | 311 | Find the best graph parameters. [] |
63c2d5de MA |
312 | |
313 | * doc/bison.texinfo | |
1a4648ff | 314 | ** Update |
c3a8cbaa | 315 | informations about ERROR_VERBOSE. [] |
1a4648ff | 316 | ** Add explainations about |
c3a8cbaa MA |
317 | skeleton muscles. [] |
318 | %skeleton. [] | |
eeeb962b | 319 | |
704a47c4 | 320 | * testsuite |
c3a8cbaa MA |
321 | ** tests/pure-parser.at [] |
322 | New tests. | |
0f8d586a AD |
323 | |
324 | * Debugging parsers | |
325 | ||
326 | From Greg McGary: | |
327 | ||
328 | akim demaille <akim.demaille@epita.fr> writes: | |
329 | ||
330 | > With great pleasure! Nonetheless, things which are debatable | |
331 | > (or not, but just `big') should be discuss in `public': something | |
332 | > like help- or bug-bison@gnu.org is just fine. Jesse and I are there, | |
333 | > but there is also Jim and some other people. | |
334 | ||
335 | I have no idea whether it qualifies as big or controversial, so I'll | |
336 | just summarize for you. I proposed this change years ago and was | |
337 | surprised that it was met with utter indifference! | |
338 | ||
339 | This debug feature is for the programs/grammars one develops with | |
340 | bison, not for debugging bison itself. I find that the YYDEBUG | |
341 | output comes in a very inconvenient format for my purposes. | |
342 | When debugging gcc, for instance, what I want is to see a trace of | |
343 | the sequence of reductions and the line#s for the semantic actions | |
344 | so I can follow what's happening. Single-step in gdb doesn't cut it | |
345 | because to move from one semantic action to the next takes you through | |
346 | lots of internal machinery of the parser, which is uninteresting. | |
347 | ||
348 | The change I made was to the format of the debug output, so that it | |
349 | comes out in the format of C error messages, digestible by emacs | |
350 | compile mode, like so: | |
351 | ||
352 | grammar.y:1234: foo: bar(0x123456) baz(0x345678) | |
353 | ||
354 | where "foo: bar baz" is the reduction rule, whose semantic action | |
355 | appears on line 1234 of the bison grammar file grammar.y. The hex | |
356 | numbers on the rhs tokens are the parse-stack values associated with | |
357 | those tokens. Of course, yytype might be something totally | |
358 | incompatible with that representation, but for the most part, yytype | |
359 | values are single words (scalars or pointers). In the case of gcc, | |
360 | they're most often pointers to tree nodes. Come to think of it, the | |
361 | right thing to do is to make the printing of stack values be | |
362 | user-definable. It would also be useful to include the filename & | |
363 | line# of the file being parsed, but the main filename & line# should | |
364 | continue to be that of grammar.y | |
365 | ||
366 | Anyway, this feature has saved my life on numerous occasions. The way | |
367 | I customarily use it is to first run bison with the traces on, isolate | |
368 | the sequence of reductions that interests me, put those traces in a | |
369 | buffer and force it into compile-mode, then visit each of those lines | |
370 | in the grammar and set breakpoints with C-x SPACE. Then, I can run | |
371 | again under the control of gdb and stop at each semantic action. | |
372 | With the hex addresses of tree nodes, I can inspect the values | |
373 | associated with any rhs token. | |
374 | ||
375 | You like? | |
cd6a695e AD |
376 | |
377 | * input synclines | |
378 | Some users create their foo.y files, and equip them with #line. Bison | |
379 | should recognize these, and preserve them. | |
0e95c1dd AD |
380 | |
381 | * BTYacc | |
382 | See if we can integrate backtracking in Bison. Contact the BTYacc | |
383 | maintainers. | |
384 | ||
385 | * Automaton report | |
386 | Display more clearly the lookaheads for each item. | |
387 | ||
388 | * RR conflicts | |
389 | See if we can use precedence between rules to solve RR conflicts. See | |
390 | what POSIX says. | |
391 | ||
392 | * Precedence | |
393 | It is unfortunate that there is a total order for precedence. It | |
394 | makes it impossible to have modular precedence information. We should | |
395 | move to partial orders. | |
396 | ||
3c9160d9 AD |
397 | This will be possible with a Bison parser for the grammar, as it will |
398 | make it much easier to extend the grammar. | |
399 | ||
0e95c1dd | 400 | * Parsing grammars |
3c9160d9 | 401 | Rewrite the reader in Flex/Bison. There will be delicate parts, in |
9306c70c AD |
402 | particular, expect the scanner to be hard to write. Many interesting |
403 | features cannot be implemented without such a new reader. | |
f294a2c2 | 404 | |
20c37f21 AD |
405 | * Presentation of the report file |
406 | From: "Baum, Nathan I" <s0009525@chelt.ac.uk> | |
407 | Subject: Token Alias Bug | |
408 | To: "'bug-bison@gnu.org'" <bug-bison@gnu.org> | |
409 | ||
410 | I've also noticed something, that whilst not *wrong*, is inconvienient: I | |
411 | use the verbose mode to help find the causes of unresolved shift/reduce | |
412 | conflicts. However, this mode insists on starting the .output file with a | |
413 | list of *resolved* conflicts, something I find quite useless. Might it be | |
414 | possible to define a -v mode, and a -vv mode -- Where the -vv mode shows | |
415 | everything, but the -v mode only tells you what you need for examining | |
416 | conflicts? (Or, perhaps, a "*** This state has N conflicts ***" marker above | |
417 | each state with conflicts.) | |
418 | ||
69991a58 AD |
419 | * $undefined |
420 | From Hans: | |
421 | - If the Bison generated parser experiences an undefined number in the | |
422 | character range, that character is written out in diagnostic messages, an | |
423 | addition to the $undefined value. | |
424 | ||
425 | Suggest: Change the name $undefined to undefined; looks better in outputs. | |
426 | ||
427 | * Default Action | |
428 | From Hans: | |
429 | - For use with my C++ parser, I transported the "switch (yyn)" statement | |
430 | that Bison writes to the bison.simple skeleton file. This way, I can remove | |
431 | the current default rule $$ = $1 implementation, which causes a double | |
432 | assignment to $$ which may not be OK under C++, replacing it with a | |
433 | "default:" part within the switch statement. | |
434 | ||
435 | Note that the default rule $$ = $1, when typed, is perfectly OK under C, | |
436 | but in the C++ implementation I made, this rule is different from | |
437 | $<type_name>$ = $<type_name>1. I therefore think that one should implement | |
438 | a Bison option where every typed default rule is explicitly written out | |
439 | (same typed ruled can of course be grouped together). | |
440 | ||
3c9160d9 AD |
441 | Note: Robert Anisko handles this. He knows how to do it. |
442 | ||
443 | * Documenting C++ output | |
444 | Write a first documentation for C++ output. | |
445 | ||
0164db68 AD |
446 | * Warnings |
447 | It would be nice to have warning support. See how Autoconf handles | |
448 | them, it is fairly well described there. It would be very nice to | |
449 | implement this in such a way that other programs could use | |
450 | lib/warnings.[ch]. | |
451 | ||
9306c70c AD |
452 | Don't work on this without first announcing you do, as I already have |
453 | thought about it, and know many of the components that can be used to | |
454 | implement it. | |
455 | ||
69991a58 AD |
456 | * Pre and post actions. |
457 | From: Florian Krohm <florian@edamail.fishkill.ibm.com> | |
458 | Subject: YYACT_EPILOGUE | |
459 | To: bug-bison@gnu.org | |
460 | X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago | |
461 | ||
462 | The other day I had the need for explicitly building the parse tree. I | |
463 | used %locations for that and defined YYLLOC_DEFAULT to call a function | |
464 | that returns the tree node for the production. Easy. But I also needed | |
465 | to assign the S-attribute to the tree node. That cannot be done in | |
466 | YYLLOC_DEFAULT, because it is invoked before the action is executed. | |
467 | The way I solved this was to define a macro YYACT_EPILOGUE that would | |
468 | be invoked after the action. For reasons of symmetry I also added | |
469 | YYACT_PROLOGUE. Although I had no use for that I can envision how it | |
470 | might come in handy for debugging purposes. | |
76551463 | 471 | All is needed is to add |
69991a58 AD |
472 | |
473 | #if YYLSP_NEEDED | |
474 | YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen)); | |
475 | #else | |
476 | YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen); | |
477 | #endif | |
478 | ||
479 | at the proper place to bison.simple. Ditto for YYACT_PROLOGUE. | |
480 | ||
481 | I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE | |
482 | to bison. If you're interested, I'll work on a patch. | |
483 | ||
f294a2c2 AD |
484 | ----- |
485 | ||
486 | Copyright (C) 2001, 2002 Free Software Foundation, Inc. | |
487 | ||
488 | This file is part of GNU Autoconf. | |
489 | ||
490 | GNU Autoconf is free software; you can redistribute it and/or modify | |
491 | it under the terms of the GNU General Public License as published by | |
492 | the Free Software Foundation; either version 2, or (at your option) | |
493 | any later version. | |
494 | ||
495 | GNU Autoconf is distributed in the hope that it will be useful, | |
496 | but WITHOUT ANY WARRANTY; without even the implied warranty of | |
497 | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
498 | GNU General Public License for more details. | |
499 | ||
500 | You should have received a copy of the GNU General Public License | |
501 | along with autoconf; see the file COPYING. If not, write to | |
502 | the Free Software Foundation, Inc., 59 Temple Place - Suite 330, | |
503 | Boston, MA 02111-1307, USA. |