]> git.saurik.com Git - bison.git/blame - TODO
calc++: factor for other extracted tests.
[bison.git] / TODO
CommitLineData
416bd7a9
MA
1-*- outline -*-
2
dd704c35 3* Short term
5ad90d52
AD
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
1e034807
AD
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
dd704c35
AD
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
42f832d6 65* Various
865f1e9f
AD
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
42f832d6
AD
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
90462b8d
AD
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
fc2476c7
AD
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
42f832d6
AD
118** YYFAIL
119It is seems to be *really* obsolete now, shall we remove it?
120
27cb5b59
AD
121** yychar == yyempty_
122The code in yyerrlab reads:
123
124 if (yychar <= YYEOF)
e9690142
JD
125 {
126 /* Return failure if at end of input. */
127 if (yychar == YYEOF)
128 YYABORT;
129 }
27cb5b59
AD
130
131There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF.
132But I can't produce the situation where yychar is YYEMPTY here, is it
133really possible? The test suite does not exercise this case.
134
135This shows that it would be interesting to manage to install skeleton
136coverage analysis to the test suite.
42f832d6 137
a2e3fa77
AD
138** Table definitions
139It should be very easy to factor the definition of the various tables,
140including the separation bw declaration and definition. See for
141instance b4_table_define in lalr1.cc. This way, we could even factor
142C vs. C++ definitions.
143
00a8a083
AD
144* From lalr1.cc to yacc.c
145** Single stack
146Merging the three stacks in lalr1.cc simplified the code, prompted for
147other improvements and also made it faster (probably because memory
148management is performed once instead of three times). I suggest that
149we do the same in yacc.c.
150
dada3cd1 151** yysyntax_error
2b008529
AD
152The code bw glr.c and yacc.c is really alike, we can certainly factor
153some parts.
dada3cd1 154
3c146b5e
AD
155* Header guards
156
32f0598d 157From Franc,ois: should we keep the directory part in the CPP guard?
3c146b5e
AD
158
159
c19988b7
AD
160* Yacc.c: CPP Macros
161
162Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite?
163They should not: it is not documented. But if they need to, let's
164find something clean (not like YYLSP_NEEDED...).
165
166
5d278082
PE
167* Installation
168
88bce5a2 169* Documentation
959e5f51
AD
170Before releasing, make sure the documentation ("Understanding your
171parser") refers to the current `output' format.
88bce5a2 172
2ab9a04f 173* Report
ec3bc396 174
c981ce9b
AD
175** Figures
176Some statistics about the grammar and the parser would be useful,
177especially when asking the user to send some information about the
178grammars she is working on. We should probably also include some
179information about the variables (I'm not sure for instance we even
180specify what LR variant was used).
181
2ab9a04f
AD
182** GLR
183How would Paul like to display the conflicted actions? In particular,
742e4900 184what when two reductions are possible on a given lookahead token, but one is
2ab9a04f
AD
185part of $default. Should we make the two reductions explicit, or just
186keep $default? See the following point.
d7215705 187
2ab9a04f
AD
188** Disabled Reductions
189See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
190what we want to do.
d7215705 191
2ab9a04f 192** Documentation
bc933ef1
AD
193Extend with error productions. The hard part will probably be finding
194the right rule so that a single state does not exhibit too many yet
195undocumented ``features''. Maybe an empty action ought to be
196presented too. Shall we try to make a single grammar with all these
197features, or should we have several very small grammars?
ec3bc396 198
2ab9a04f
AD
199** --report=conflict-path
200Provide better assistance for understanding the conflicts by providing
201a sample text exhibiting the (LALR) ambiguity. See the paper from
202DeRemer and Penello: they already provide the algorithm.
203
38eb7751
PE
204** Statically check for potential ambiguities in GLR grammars. See
205<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
206
ec3bc396 207
948be909 208* Extensions
2ab9a04f 209
959e5f51
AD
210** $-1
211We should find a means to provide an access to values deep in the
212stack. For instance, instead of
213
e9690142 214 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
959e5f51
AD
215
216we should be able to have:
217
218 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
219
220Or something like this.
221
f0e48240
AD
222** %if and the like
223It should be possible to have %if/%else/%endif. The implementation is
224not clear: should it be lexical or syntactic. Vadim Maslow thinks it
225must be in the scanner: we must not parse what is in a switched off
226part of %if. Akim Demaille thinks it should be in the parser, so as
227to avoid falling into another CPP mistake.
228
ca752c34
AD
229** XML Output
230There are couple of available extensions of Bison targeting some XML
231output. Some day we should consider including them. One issue is
232that they seem to be quite orthogonal to the parsing technique, and
233seem to depend mostly on the possibility to have some code triggered
234for each reduction. As a matter of fact, such hooks could also be
235used to generate the yydebug traces. Some generic scheme probably
236exists in there.
237
238XML output for GNU Bison and gcc
239 http://www.cs.may.ie/~jpower/Research/bisonXML/
240
241XML output for GNU Bison
242 http://yaxx.sourceforge.net/
f0e48240 243
fa770c86
AD
244* Unit rules
245Maybe we could expand unit rules, i.e., transform
246
e9690142
JD
247 exp: arith | bool;
248 arith: exp '+' exp;
249 bool: exp '&' exp;
fa770c86
AD
250
251into
252
e9690142 253 exp: exp '+' exp | exp '&' exp;
fa770c86
AD
254
255when there are no actions. This can significantly speed up some
d7215705
AD
256grammars. I can't find the papers. In particular the book `LR
257parsing: Theory and Practice' is impossible to find, but according to
258`Parsing Techniques: a Practical Guide', it includes information about
259this issue. Does anybody have it?
fa770c86 260
51dec47b 261
51dec47b 262
2ab9a04f 263* Documentation
51dec47b 264
2ab9a04f
AD
265** History/Bibliography
266Some history of Bison and some bibliography would be most welcome.
267Are there any Texinfo standards for bibliography?
268
fa819509
AD
269** %printer
270Wow, %printer is not documented. Clearly mark YYPRINT as obsolete.
2ab9a04f 271
948be909
PE
272* Java, Fortran, etc.
273
948be909 274
2ab9a04f
AD
275* Coding system independence
276Paul notes:
277
e9690142
JD
278 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
279 255). It also assumes that the 8-bit character encoding is
280 the same for the invocation of 'bison' as it is for the
281 invocation of 'cc', but this is not necessarily true when
282 people run bison on an ASCII host and then use cc on an EBCDIC
283 host. I don't think these topics are worth our time
284 addressing (unless we find a gung-ho volunteer for EBCDIC or
285 PDP-10 ports :-) but they should probably be documented
286 somewhere.
287
288 More importantly, Bison does not currently allow NUL bytes in
289 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
290 the source code. This should get fixed.
aef1ffd5 291
bcb05e75 292* --graph
45567173 293Show reductions.
bcb05e75 294
704a47c4 295* Broken options ?
45567173
AD
296** %token-table
297** Skeleton strategy
728c4be2 298Must we keep %token-table?
416bd7a9 299
0e95c1dd 300* BTYacc
f0e48240 301See if we can integrate backtracking in Bison. Charles-Henri de
df72984a
AD
302Boysson <de-boy_c@epita.fr> has been working on this, but never gave
303the results.
304
305Vadim Maslow, the maintainer of BTYacc was once contacted. Adjusting
306the Bison grammar parser will be needed to support some extra BTYacc
307features. This is less urgent.
0e95c1dd 308
2ab9a04f
AD
309** Keeping the conflicted actions
310First, analyze the differences between byacc and btyacc (I'm referring
311to the executables). Find where the conflicts are preserved.
312
313** Compare with the GLR tables
948be909 314See how isomorphic the way BTYacc and the way the GLR adjustments in
2ab9a04f
AD
315Bison are compatible. *As much as possible* one should try to use the
316same implementation in the Bison executables. I insist: it should be
317very feasible to use the very same conflict tables.
318
319** Adjust the skeletons
320Import the skeletons for C and C++.
321
0e95c1dd
AD
322
323* Precedence
2ab9a04f
AD
324
325** Partial order
0e95c1dd
AD
326It is unfortunate that there is a total order for precedence. It
327makes it impossible to have modular precedence information. We should
2ab9a04f 328move to partial orders (sounds like series/parallel orders to me).
0e95c1dd 329
2ab9a04f
AD
330** RR conflicts
331See if we can use precedence between rules to solve RR conflicts. See
332what POSIX says.
333
334
69991a58
AD
335* $undefined
336From Hans:
337- If the Bison generated parser experiences an undefined number in the
338character range, that character is written out in diagnostic messages, an
339addition to the $undefined value.
340
341Suggest: Change the name $undefined to undefined; looks better in outputs.
342
2ab9a04f 343
69991a58
AD
344* Default Action
345From Hans:
346- For use with my C++ parser, I transported the "switch (yyn)" statement
347that Bison writes to the bison.simple skeleton file. This way, I can remove
348the current default rule $$ = $1 implementation, which causes a double
349assignment to $$ which may not be OK under C++, replacing it with a
350"default:" part within the switch statement.
351
352Note that the default rule $$ = $1, when typed, is perfectly OK under C,
353but in the C++ implementation I made, this rule is different from
354$<type_name>$ = $<type_name>1. I therefore think that one should implement
355a Bison option where every typed default rule is explicitly written out
356(same typed ruled can of course be grouped together).
357
358* Pre and post actions.
359From: Florian Krohm <florian@edamail.fishkill.ibm.com>
360Subject: YYACT_EPILOGUE
361To: bug-bison@gnu.org
362X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
363
364The other day I had the need for explicitly building the parse tree. I
365used %locations for that and defined YYLLOC_DEFAULT to call a function
366that returns the tree node for the production. Easy. But I also needed
367to assign the S-attribute to the tree node. That cannot be done in
368YYLLOC_DEFAULT, because it is invoked before the action is executed.
369The way I solved this was to define a macro YYACT_EPILOGUE that would
370be invoked after the action. For reasons of symmetry I also added
371YYACT_PROLOGUE. Although I had no use for that I can envision how it
372might come in handy for debugging purposes.
76551463 373All is needed is to add
69991a58
AD
374
375#if YYLSP_NEEDED
376 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
377#else
378 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
379#endif
380
381at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
382
383I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
384to bison. If you're interested, I'll work on a patch.
385
35fe0834
PE
386* Better graphics
387Equip the parser with a means to create the (visual) parse tree.
d7215705 388
3c5362b8
JD
389* Complaint submessage indentation.
390We already have an implementation that works fairly well for named
391reference messages, but it would be nice to use it consistently for all
392submessages from Bison. For example, the "previous definition"
393submessage or the list of correct values for a %define variable might
394look better with indentation.
395
396However, the current implementation makes the assumption that the
397location printed on the first line is not usually much shorter than the
398locations printed on the submessage lines that follow. That assumption
399may not hold true as often for some kinds of submessages especially if
400we ever support multiple grammar files.
401
402Here's a proposal for how a new implementation might look:
403
404 http://lists.gnu.org/archive/html/bison-patches/2009-09/msg00086.html
405
f294a2c2
AD
406-----
407
34136e65 408Copyright (C) 2001-2004, 2006, 2008-2012 Free Software Foundation, Inc.
f294a2c2 409
51cbef6f 410This file is part of Bison, the GNU Compiler Compiler.
f294a2c2 411
f16b0819 412This program is free software: you can redistribute it and/or modify
f294a2c2 413it under the terms of the GNU General Public License as published by
f16b0819
PE
414the Free Software Foundation, either version 3 of the License, or
415(at your option) any later version.
f294a2c2 416
f16b0819 417This program is distributed in the hope that it will be useful,
f294a2c2
AD
418but WITHOUT ANY WARRANTY; without even the implied warranty of
419MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
420GNU General Public License for more details.
421
422You should have received a copy of the GNU General Public License
f16b0819 423along with this program. If not, see <http://www.gnu.org/licenses/>.