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