]>
Commit | Line | Data |
---|---|---|
1 | -*- outline -*- | |
2 | ||
3 | * Short term | |
4 | ** Use syntax_error from the scanner? | |
5 | This would provide a means to raise syntax error from function called | |
6 | from the scanner. Actually, there is no good solution to report a | |
7 | lexical error in general. Usually they are kept at the scanner level | |
8 | only, ignoring the guilty token. But that might not be the best bet, | |
9 | since we don't benefit from the syntactic error recovery. | |
10 | ||
11 | We still have the possibility to return an invalid token number, which | |
12 | does the trick. But then the error message from the parser is poor | |
13 | (something like "unexpected $undefined"). Since the scanner probably | |
14 | already reported the error, we should directly enter error-recovery, | |
15 | without reporting the error message (i.e., YYERROR's semantics). | |
16 | ||
17 | Back to lalr1.cc (whose name is now quite unfortunate, since it also | |
18 | covers lr and ielr), if we support exceptions from yylex, should we | |
19 | propose a lexical_error in addition to syntax_error? Should they have | |
20 | a common root, say parse_error? Should syntax_error be renamed | |
21 | syntactic_error for consistency with lexical_error? | |
22 | ||
23 | ** Variable names. | |
24 | What should we name `variant' and `lex_symbol'? | |
25 | ||
26 | ** Use b4_symbol in all the skeleton | |
27 | Then remove the older system, including the tables generated by | |
28 | output.c | |
29 | ||
30 | ** Update the documentation on gnu.org | |
31 | ||
32 | ** Get rid of fake #lines [Bison: ...] | |
33 | Possibly as simple as checking whether the column number is nonnegative. | |
34 | ||
35 | I have seen messages like the following from GCC. | |
36 | ||
37 | <built-in>:0: fatal error: opening dependency file .deps/libltdl/argz.Tpo: No such file or directory | |
38 | ||
39 | ||
40 | ** Discuss about %printer/%destroy in the case of C++. | |
41 | It would be very nice to provide the symbol classes with an operator<< | |
42 | and a destructor. Unfortunately the syntax we have chosen for | |
43 | %destroy and %printer make them hard to reuse. For instance, the user | |
44 | is invited to write something like | |
45 | ||
46 | %printer { debug_stream() << $$; } <my_type>; | |
47 | ||
48 | which is hard to reuse elsewhere since it wants to use | |
49 | "debug_stream()" to find the stream to use. The same applies to | |
50 | %destroy: we told the user she could use the members of the Parser | |
51 | class in the printers/destructors, which is not good for an operator<< | |
52 | since it is no longer bound to a particular parser, it's just a | |
53 | (standalone symbol). | |
54 | ||
55 | ** Rename LR0.cc | |
56 | as lr0.cc, why upper case? | |
57 | ||
58 | ** bench several bisons. | |
59 | Enhance bench.pl with %b to run different bisons. | |
60 | ||
61 | ** Use b4_symbol everywhere. | |
62 | Move its definition in the more standard places and deploy it in other | |
63 | skeletons. | |
64 | ||
65 | * Various | |
66 | ** YYPRINT | |
67 | glr.c inherits its symbol_print function from c.m4, which supports | |
68 | YYPRINT. But to use YYPRINT yytoknum is needed, which not defined by | |
69 | glr.c. | |
70 | ||
71 | Anyway, IMHO YYPRINT is obsolete and should be restricted to yacc.c. | |
72 | ||
73 | ** YYERRCODE | |
74 | Defined to 256, but not used, not documented. Probably the token | |
75 | number for the error token, which POSIX wants to be 256, but which | |
76 | Bison might renumber if the user used number 256. Keep fix and doc? | |
77 | Throw away? | |
78 | ||
79 | Also, why don't we output the token name of the error token in the | |
80 | output? It is explicitly skipped: | |
81 | ||
82 | /* Skip error token and tokens without identifier. */ | |
83 | if (sym != errtoken && id) | |
84 | ||
85 | Of course there are issues with name spaces, but if we disable we have | |
86 | something which seems to be more simpler and more consistent instead | |
87 | of the special case YYERRCODE. | |
88 | ||
89 | enum yytokentype { | |
90 | error = 256, | |
91 | // ... | |
92 | }; | |
93 | ||
94 | ||
95 | We could (should?) also treat the case of the undef_token, which is | |
96 | numbered 257 for yylex, and 2 internal. Both appear for instance in | |
97 | toknum: | |
98 | ||
99 | const unsigned short int | |
100 | parser::yytoken_number_[] = | |
101 | { | |
102 | 0, 256, 257, 258, 259, 260, 261, 262, 263, 264, | |
103 | ||
104 | while here | |
105 | ||
106 | enum yytokentype { | |
107 | TOK_EOF = 0, | |
108 | TOK_EQ = 258, | |
109 | ||
110 | so both 256 and 257 are "mysterious". | |
111 | ||
112 | const char* | |
113 | const parser::yytname_[] = | |
114 | { | |
115 | "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"", | |
116 | ||
117 | ||
118 | ** YYFAIL | |
119 | It is seems to be *really* obsolete now, shall we remove it? | |
120 | ||
121 | ** YYBACKUP | |
122 | There is no test about it, no examples in the doc, and I'm not sure | |
123 | what it should look like. For instance what follows crashes. | |
124 | ||
125 | %error-verbose | |
126 | %debug | |
127 | %pure-parser | |
128 | %code { | |
129 | # include <stdio.h> | |
130 | # include <stdlib.h> | |
131 | # include <assert.h> | |
132 | ||
133 | static void yyerror (const char *msg); | |
134 | static int yylex (YYSTYPE *yylval); | |
135 | } | |
136 | %% | |
137 | exp: | |
138 | 'a' { printf ("a: %d\n", $1); } | |
139 | | 'b' { YYBACKUP('a', 123); } | |
140 | ; | |
141 | %% | |
142 | static int | |
143 | yylex (YYSTYPE *yylval) | |
144 | { | |
145 | static char const input[] = "b"; | |
146 | static size_t toknum; | |
147 | assert (toknum < sizeof input); | |
148 | *yylval = (toknum + 1) * 10; | |
149 | return input[toknum++]; | |
150 | } | |
151 | ||
152 | static void | |
153 | yyerror (const char *msg) | |
154 | { | |
155 | fprintf (stderr, "%s\n", msg); | |
156 | } | |
157 | ||
158 | int | |
159 | main (void) | |
160 | { | |
161 | yydebug = !!getenv("YYDEBUG"); | |
162 | return yyparse (); | |
163 | } | |
164 | ||
165 | ** yychar == yyempty_ | |
166 | The code in yyerrlab reads: | |
167 | ||
168 | if (yychar <= YYEOF) | |
169 | { | |
170 | /* Return failure if at end of input. */ | |
171 | if (yychar == YYEOF) | |
172 | YYABORT; | |
173 | } | |
174 | ||
175 | There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF. | |
176 | But I can't produce the situation where yychar is YYEMPTY here, is it | |
177 | really possible? The test suite does not exercise this case. | |
178 | ||
179 | This shows that it would be interesting to manage to install skeleton | |
180 | coverage analysis to the test suite. | |
181 | ||
182 | ** Table definitions | |
183 | It should be very easy to factor the definition of the various tables, | |
184 | including the separation bw declaration and definition. See for | |
185 | instance b4_table_define in lalr1.cc. This way, we could even factor | |
186 | C vs. C++ definitions. | |
187 | ||
188 | * From lalr1.cc to yacc.c | |
189 | ** Single stack | |
190 | Merging the three stacks in lalr1.cc simplified the code, prompted for | |
191 | other improvements and also made it faster (probably because memory | |
192 | management is performed once instead of three times). I suggest that | |
193 | we do the same in yacc.c. | |
194 | ||
195 | ** yysyntax_error | |
196 | The code bw glr.c and yacc.c is really alike, we can certainly factor | |
197 | some parts. | |
198 | ||
199 | * Header guards | |
200 | ||
201 | From Franc,ois: should we keep the directory part in the CPP guard? | |
202 | ||
203 | ||
204 | * Yacc.c: CPP Macros | |
205 | ||
206 | Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite? | |
207 | They should not: it is not documented. But if they need to, let's | |
208 | find something clean (not like YYLSP_NEEDED...). | |
209 | ||
210 | ||
211 | * Installation | |
212 | ||
213 | * Documentation | |
214 | Before releasing, make sure the documentation ("Understanding your | |
215 | parser") refers to the current `output' format. | |
216 | ||
217 | * Report | |
218 | ||
219 | ** Figures | |
220 | Some statistics about the grammar and the parser would be useful, | |
221 | especially when asking the user to send some information about the | |
222 | grammars she is working on. We should probably also include some | |
223 | information about the variables (I'm not sure for instance we even | |
224 | specify what LR variant was used). | |
225 | ||
226 | ** GLR | |
227 | How would Paul like to display the conflicted actions? In particular, | |
228 | what when two reductions are possible on a given lookahead token, but one is | |
229 | part of $default. Should we make the two reductions explicit, or just | |
230 | keep $default? See the following point. | |
231 | ||
232 | ** Disabled Reductions | |
233 | See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide | |
234 | what we want to do. | |
235 | ||
236 | ** Documentation | |
237 | Extend with error productions. The hard part will probably be finding | |
238 | the right rule so that a single state does not exhibit too many yet | |
239 | undocumented ``features''. Maybe an empty action ought to be | |
240 | presented too. Shall we try to make a single grammar with all these | |
241 | features, or should we have several very small grammars? | |
242 | ||
243 | ** --report=conflict-path | |
244 | Provide better assistance for understanding the conflicts by providing | |
245 | a sample text exhibiting the (LALR) ambiguity. See the paper from | |
246 | DeRemer and Penello: they already provide the algorithm. | |
247 | ||
248 | ** Statically check for potential ambiguities in GLR grammars. See | |
249 | <http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach. | |
250 | ||
251 | ||
252 | * Extensions | |
253 | ||
254 | ** Labeling the symbols | |
255 | Have a look at the Lemon parser generator: instead of $1, $2 etc. they | |
256 | can name the values. This is much more pleasant. For instance: | |
257 | ||
258 | exp (res): exp (a) '+' exp (b) { $res = $a + $b; }; | |
259 | ||
260 | I love this. I have been bitten too often by the removal of the | |
261 | symbol, and forgetting to shift all the $n to $n-1. If you are | |
262 | unlucky, it compiles... | |
263 | ||
264 | But instead of using $a etc., we can use regular variables. And | |
265 | instead of using (), I propose to use `:' (again). Paul suggests | |
266 | supporting `->' in addition to `:' to separate LHS and RHS. In other | |
267 | words: | |
268 | ||
269 | r:exp -> a:exp '+' b:exp { r = a + b; }; | |
270 | ||
271 | That requires an significant improvement of the grammar parser. Using | |
272 | GLR would be nice. It also requires that Bison know the type of the | |
273 | symbols (which will be useful for %include anyway). So we have some | |
274 | time before... | |
275 | ||
276 | Note that there remains the problem of locations: `@r'? | |
277 | ||
278 | ||
279 | ** $-1 | |
280 | We should find a means to provide an access to values deep in the | |
281 | stack. For instance, instead of | |
282 | ||
283 | baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; } | |
284 | ||
285 | we should be able to have: | |
286 | ||
287 | foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; } | |
288 | ||
289 | Or something like this. | |
290 | ||
291 | ** %if and the like | |
292 | It should be possible to have %if/%else/%endif. The implementation is | |
293 | not clear: should it be lexical or syntactic. Vadim Maslow thinks it | |
294 | must be in the scanner: we must not parse what is in a switched off | |
295 | part of %if. Akim Demaille thinks it should be in the parser, so as | |
296 | to avoid falling into another CPP mistake. | |
297 | ||
298 | ** XML Output | |
299 | There are couple of available extensions of Bison targeting some XML | |
300 | output. Some day we should consider including them. One issue is | |
301 | that they seem to be quite orthogonal to the parsing technique, and | |
302 | seem to depend mostly on the possibility to have some code triggered | |
303 | for each reduction. As a matter of fact, such hooks could also be | |
304 | used to generate the yydebug traces. Some generic scheme probably | |
305 | exists in there. | |
306 | ||
307 | XML output for GNU Bison and gcc | |
308 | http://www.cs.may.ie/~jpower/Research/bisonXML/ | |
309 | ||
310 | XML output for GNU Bison | |
311 | http://yaxx.sourceforge.net/ | |
312 | ||
313 | * Unit rules | |
314 | Maybe we could expand unit rules, i.e., transform | |
315 | ||
316 | exp: arith | bool; | |
317 | arith: exp '+' exp; | |
318 | bool: exp '&' exp; | |
319 | ||
320 | into | |
321 | ||
322 | exp: exp '+' exp | exp '&' exp; | |
323 | ||
324 | when there are no actions. This can significantly speed up some | |
325 | grammars. I can't find the papers. In particular the book `LR | |
326 | parsing: Theory and Practice' is impossible to find, but according to | |
327 | `Parsing Techniques: a Practical Guide', it includes information about | |
328 | this issue. Does anybody have it? | |
329 | ||
330 | ||
331 | ||
332 | * Documentation | |
333 | ||
334 | ** History/Bibliography | |
335 | Some history of Bison and some bibliography would be most welcome. | |
336 | Are there any Texinfo standards for bibliography? | |
337 | ||
338 | ** %printer | |
339 | Wow, %printer is not documented. Clearly mark YYPRINT as obsolete. | |
340 | ||
341 | * Java, Fortran, etc. | |
342 | ||
343 | ||
344 | * Coding system independence | |
345 | Paul notes: | |
346 | ||
347 | Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is | |
348 | 255). It also assumes that the 8-bit character encoding is | |
349 | the same for the invocation of 'bison' as it is for the | |
350 | invocation of 'cc', but this is not necessarily true when | |
351 | people run bison on an ASCII host and then use cc on an EBCDIC | |
352 | host. I don't think these topics are worth our time | |
353 | addressing (unless we find a gung-ho volunteer for EBCDIC or | |
354 | PDP-10 ports :-) but they should probably be documented | |
355 | somewhere. | |
356 | ||
357 | More importantly, Bison does not currently allow NUL bytes in | |
358 | tokens, either via escapes (e.g., "x\0y") or via a NUL byte in | |
359 | the source code. This should get fixed. | |
360 | ||
361 | * --graph | |
362 | Show reductions. | |
363 | ||
364 | * Broken options ? | |
365 | ** %token-table | |
366 | ** Skeleton strategy | |
367 | Must we keep %token-table? | |
368 | ||
369 | * BTYacc | |
370 | See if we can integrate backtracking in Bison. Charles-Henri de | |
371 | Boysson <de-boy_c@epita.fr> has been working on this, but never gave | |
372 | the results. | |
373 | ||
374 | Vadim Maslow, the maintainer of BTYacc was once contacted. Adjusting | |
375 | the Bison grammar parser will be needed to support some extra BTYacc | |
376 | features. This is less urgent. | |
377 | ||
378 | ** Keeping the conflicted actions | |
379 | First, analyze the differences between byacc and btyacc (I'm referring | |
380 | to the executables). Find where the conflicts are preserved. | |
381 | ||
382 | ** Compare with the GLR tables | |
383 | See how isomorphic the way BTYacc and the way the GLR adjustments in | |
384 | Bison are compatible. *As much as possible* one should try to use the | |
385 | same implementation in the Bison executables. I insist: it should be | |
386 | very feasible to use the very same conflict tables. | |
387 | ||
388 | ** Adjust the skeletons | |
389 | Import the skeletons for C and C++. | |
390 | ||
391 | ||
392 | * Precedence | |
393 | ||
394 | ** Partial order | |
395 | It is unfortunate that there is a total order for precedence. It | |
396 | makes it impossible to have modular precedence information. We should | |
397 | move to partial orders (sounds like series/parallel orders to me). | |
398 | ||
399 | ** RR conflicts | |
400 | See if we can use precedence between rules to solve RR conflicts. See | |
401 | what POSIX says. | |
402 | ||
403 | ||
404 | * $undefined | |
405 | From Hans: | |
406 | - If the Bison generated parser experiences an undefined number in the | |
407 | character range, that character is written out in diagnostic messages, an | |
408 | addition to the $undefined value. | |
409 | ||
410 | Suggest: Change the name $undefined to undefined; looks better in outputs. | |
411 | ||
412 | ||
413 | * Default Action | |
414 | From Hans: | |
415 | - For use with my C++ parser, I transported the "switch (yyn)" statement | |
416 | that Bison writes to the bison.simple skeleton file. This way, I can remove | |
417 | the current default rule $$ = $1 implementation, which causes a double | |
418 | assignment to $$ which may not be OK under C++, replacing it with a | |
419 | "default:" part within the switch statement. | |
420 | ||
421 | Note that the default rule $$ = $1, when typed, is perfectly OK under C, | |
422 | but in the C++ implementation I made, this rule is different from | |
423 | $<type_name>$ = $<type_name>1. I therefore think that one should implement | |
424 | a Bison option where every typed default rule is explicitly written out | |
425 | (same typed ruled can of course be grouped together). | |
426 | ||
427 | * Pre and post actions. | |
428 | From: Florian Krohm <florian@edamail.fishkill.ibm.com> | |
429 | Subject: YYACT_EPILOGUE | |
430 | To: bug-bison@gnu.org | |
431 | X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago | |
432 | ||
433 | The other day I had the need for explicitly building the parse tree. I | |
434 | used %locations for that and defined YYLLOC_DEFAULT to call a function | |
435 | that returns the tree node for the production. Easy. But I also needed | |
436 | to assign the S-attribute to the tree node. That cannot be done in | |
437 | YYLLOC_DEFAULT, because it is invoked before the action is executed. | |
438 | The way I solved this was to define a macro YYACT_EPILOGUE that would | |
439 | be invoked after the action. For reasons of symmetry I also added | |
440 | YYACT_PROLOGUE. Although I had no use for that I can envision how it | |
441 | might come in handy for debugging purposes. | |
442 | All is needed is to add | |
443 | ||
444 | #if YYLSP_NEEDED | |
445 | YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen)); | |
446 | #else | |
447 | YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen); | |
448 | #endif | |
449 | ||
450 | at the proper place to bison.simple. Ditto for YYACT_PROLOGUE. | |
451 | ||
452 | I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE | |
453 | to bison. If you're interested, I'll work on a patch. | |
454 | ||
455 | * Better graphics | |
456 | Equip the parser with a means to create the (visual) parse tree. | |
457 | ||
458 | * Complaint submessage indentation. | |
459 | We already have an implementation that works fairly well for named | |
460 | reference messages, but it would be nice to use it consistently for all | |
461 | submessages from Bison. For example, the "previous definition" | |
462 | submessage or the list of correct values for a %define variable might | |
463 | look better with indentation. | |
464 | ||
465 | However, the current implementation makes the assumption that the | |
466 | location printed on the first line is not usually much shorter than the | |
467 | locations printed on the submessage lines that follow. That assumption | |
468 | may not hold true as often for some kinds of submessages especially if | |
469 | we ever support multiple grammar files. | |
470 | ||
471 | Here's a proposal for how a new implementation might look: | |
472 | ||
473 | http://lists.gnu.org/archive/html/bison-patches/2009-09/msg00086.html | |
474 | ||
475 | ----- | |
476 | ||
477 | Copyright (C) 2001-2004, 2006, 2008-2010 Free Software Foundation, Inc. | |
478 | ||
479 | This file is part of Bison, the GNU Compiler Compiler. | |
480 | ||
481 | This program is free software: you can redistribute it and/or modify | |
482 | it under the terms of the GNU General Public License as published by | |
483 | the Free Software Foundation, either version 3 of the License, or | |
484 | (at your option) any later version. | |
485 | ||
486 | This program is distributed in the hope that it will be useful, | |
487 | but WITHOUT ANY WARRANTY; without even the implied warranty of | |
488 | MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the | |
489 | GNU General Public License for more details. | |
490 | ||
491 | You should have received a copy of the GNU General Public License | |
492 | along with this program. If not, see <http://www.gnu.org/licenses/>. |