]> git.saurik.com Git - bison.git/blame - TODO
skeletons: no longer define YYLSP_NEEDED.
[bison.git] / TODO
CommitLineData
ff1b7a13
AD
1* Short term
2** Variable names.
3What should we name `variant' and `lex_symbol'?
4
5** Use b4_symbol in all the skeleton
6Move its definition in the more standard places and deploy it in other
7skeletons. Then remove the older system, including the tables
8generated by output.c
9
10** Update the documentation on gnu.org
11
12** Get rid of fake #lines [Bison: ...]
13Possibly as simple as checking whether the column number is nonnegative.
14
15I have seen messages like the following from GCC.
16
17<built-in>:0: fatal error: opening dependency file .deps/libltdl/argz.Tpo: No such file or directory
18
19
20** Discuss about %printer/%destroy in the case of C++.
21It would be very nice to provide the symbol classes with an operator<<
22and a destructor. Unfortunately the syntax we have chosen for
23%destroy and %printer make them hard to reuse. For instance, the user
24is invited to write something like
25
26 %printer { debug_stream() << $$; } <my_type>;
27
28which is hard to reuse elsewhere since it wants to use
29"debug_stream()" to find the stream to use. The same applies to
30%destroy: we told the user she could use the members of the Parser
31class in the printers/destructors, which is not good for an operator<<
32since it is no longer bound to a particular parser, it's just a
33(standalone symbol).
34
35** Rename LR0.cc
36as lr0.cc, why upper case?
37
38** bench several bisons.
39Enhance bench.pl with %b to run different bisons.
40
41* Various
42** Warnings
43Warnings about type tags that are used in printer and dtors, but not
44for symbols?
45
46** YYERRCODE
47Defined to 256, but not used, not documented. Probably the token
48number for the error token, which POSIX wants to be 256, but which
49Bison might renumber if the user used number 256. Keep fix and doc?
50Throw away?
51
52Also, why don't we output the token name of the error token in the
53output? It is explicitly skipped:
54
55 /* Skip error token and tokens without identifier. */
56 if (sym != errtoken && id)
57
58Of course there are issues with name spaces, but if we disable we have
59something which seems to be more simpler and more consistent instead
60of the special case YYERRCODE.
61
62 enum yytokentype {
63 error = 256,
64 // ...
65 };
66
67
68We could (should?) also treat the case of the undef_token, which is
69numbered 257 for yylex, and 2 internal. Both appear for instance in
70toknum:
71
72 const unsigned short int
73 parser::yytoken_number_[] =
74 {
75 0, 256, 257, 258, 259, 260, 261, 262, 263, 264,
76
77while here
78
79 enum yytokentype {
80 TOK_EOF = 0,
81 TOK_EQ = 258,
82
83so both 256 and 257 are "mysterious".
84
85 const char*
86 const parser::yytname_[] =
87 {
88 "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
89
90
91** YYFAIL
92It is seems to be *really* obsolete now, shall we remove it?
93
94** yychar == yyempty_
95The code in yyerrlab reads:
96
97 if (yychar <= YYEOF)
98 {
99 /* Return failure if at end of input. */
100 if (yychar == YYEOF)
101 YYABORT;
102 }
103
104There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF.
105But I can't produce the situation where yychar is YYEMPTY here, is it
106really possible? The test suite does not exercise this case.
107
108This shows that it would be interesting to manage to install skeleton
109coverage analysis to the test suite.
110
111** Table definitions
112It should be very easy to factor the definition of the various tables,
113including the separation bw declaration and definition. See for
114instance b4_table_define in lalr1.cc. This way, we could even factor
115C vs. C++ definitions.
116
117* From lalr1.cc to yacc.c
118** Single stack
119Merging the three stacks in lalr1.cc simplified the code, prompted for
120other improvements and also made it faster (probably because memory
121management is performed once instead of three times). I suggest that
122we do the same in yacc.c.
123
124** yysyntax_error
125The code bw glr.c and yacc.c is really alike, we can certainly factor
126some parts.
416bd7a9 127
3c146b5e 128
2ab9a04f 129* Report
ec3bc396 130
ff1b7a13
AD
131** Figures
132Some statistics about the grammar and the parser would be useful,
133especially when asking the user to send some information about the
134grammars she is working on. We should probably also include some
135information about the variables (I'm not sure for instance we even
136specify what LR variant was used).
137
2ab9a04f
AD
138** GLR
139How would Paul like to display the conflicted actions? In particular,
742e4900 140what when two reductions are possible on a given lookahead token, but one is
2ab9a04f
AD
141part of $default. Should we make the two reductions explicit, or just
142keep $default? See the following point.
d7215705 143
2ab9a04f
AD
144** Disabled Reductions
145See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
146what we want to do.
d7215705 147
2ab9a04f 148** Documentation
bc933ef1
AD
149Extend with error productions. The hard part will probably be finding
150the right rule so that a single state does not exhibit too many yet
151undocumented ``features''. Maybe an empty action ought to be
152presented too. Shall we try to make a single grammar with all these
153features, or should we have several very small grammars?
ec3bc396 154
2ab9a04f
AD
155** --report=conflict-path
156Provide better assistance for understanding the conflicts by providing
157a sample text exhibiting the (LALR) ambiguity. See the paper from
158DeRemer and Penello: they already provide the algorithm.
159
38eb7751
PE
160** Statically check for potential ambiguities in GLR grammars. See
161<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
162
ec3bc396 163
948be909 164* Extensions
2ab9a04f 165
959e5f51
AD
166** $-1
167We should find a means to provide an access to values deep in the
168stack. For instance, instead of
169
ff1b7a13 170 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
959e5f51
AD
171
172we should be able to have:
173
174 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
175
176Or something like this.
177
f0e48240
AD
178** %if and the like
179It should be possible to have %if/%else/%endif. The implementation is
180not clear: should it be lexical or syntactic. Vadim Maslow thinks it
181must be in the scanner: we must not parse what is in a switched off
182part of %if. Akim Demaille thinks it should be in the parser, so as
183to avoid falling into another CPP mistake.
184
ca752c34
AD
185** XML Output
186There are couple of available extensions of Bison targeting some XML
187output. Some day we should consider including them. One issue is
188that they seem to be quite orthogonal to the parsing technique, and
189seem to depend mostly on the possibility to have some code triggered
190for each reduction. As a matter of fact, such hooks could also be
191used to generate the yydebug traces. Some generic scheme probably
192exists in there.
193
194XML output for GNU Bison and gcc
195 http://www.cs.may.ie/~jpower/Research/bisonXML/
196
197XML output for GNU Bison
198 http://yaxx.sourceforge.net/
f0e48240 199
fa770c86
AD
200* Unit rules
201Maybe we could expand unit rules, i.e., transform
202
ff1b7a13
AD
203 exp: arith | bool;
204 arith: exp '+' exp;
205 bool: exp '&' exp;
fa770c86
AD
206
207into
208
ff1b7a13 209 exp: exp '+' exp | exp '&' exp;
fa770c86
AD
210
211when there are no actions. This can significantly speed up some
d7215705
AD
212grammars. I can't find the papers. In particular the book `LR
213parsing: Theory and Practice' is impossible to find, but according to
214`Parsing Techniques: a Practical Guide', it includes information about
215this issue. Does anybody have it?
fa770c86 216
51dec47b 217
51dec47b 218
2ab9a04f 219* Documentation
51dec47b 220
2ab9a04f
AD
221** History/Bibliography
222Some history of Bison and some bibliography would be most welcome.
223Are there any Texinfo standards for bibliography?
224
2ab9a04f
AD
225* Coding system independence
226Paul notes:
227
ff1b7a13
AD
228 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
229 255). It also assumes that the 8-bit character encoding is
230 the same for the invocation of 'bison' as it is for the
231 invocation of 'cc', but this is not necessarily true when
232 people run bison on an ASCII host and then use cc on an EBCDIC
233 host. I don't think these topics are worth our time
234 addressing (unless we find a gung-ho volunteer for EBCDIC or
235 PDP-10 ports :-) but they should probably be documented
236 somewhere.
fa770c86 237
ff1b7a13
AD
238 More importantly, Bison does not currently allow NUL bytes in
239 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
240 the source code. This should get fixed.
aef1ffd5 241
bcb05e75 242* --graph
45567173 243Show reductions.
bcb05e75 244
704a47c4 245* Broken options ?
45567173
AD
246** %token-table
247** Skeleton strategy
728c4be2 248Must we keep %token-table?
416bd7a9 249
0e95c1dd 250* Precedence
2ab9a04f
AD
251
252** Partial order
0e95c1dd
AD
253It is unfortunate that there is a total order for precedence. It
254makes it impossible to have modular precedence information. We should
2ab9a04f 255move to partial orders (sounds like series/parallel orders to me).
0e95c1dd 256
2ab9a04f
AD
257** RR conflicts
258See if we can use precedence between rules to solve RR conflicts. See
259what POSIX says.
260
261
69991a58
AD
262* $undefined
263From Hans:
264- If the Bison generated parser experiences an undefined number in the
265character range, that character is written out in diagnostic messages, an
266addition to the $undefined value.
267
268Suggest: Change the name $undefined to undefined; looks better in outputs.
269
2ab9a04f 270
69991a58
AD
271* Default Action
272From Hans:
273- For use with my C++ parser, I transported the "switch (yyn)" statement
274that Bison writes to the bison.simple skeleton file. This way, I can remove
275the current default rule $$ = $1 implementation, which causes a double
276assignment to $$ which may not be OK under C++, replacing it with a
277"default:" part within the switch statement.
278
279Note that the default rule $$ = $1, when typed, is perfectly OK under C,
280but in the C++ implementation I made, this rule is different from
281$<type_name>$ = $<type_name>1. I therefore think that one should implement
282a Bison option where every typed default rule is explicitly written out
283(same typed ruled can of course be grouped together).
284
285* Pre and post actions.
286From: Florian Krohm <florian@edamail.fishkill.ibm.com>
287Subject: YYACT_EPILOGUE
288To: bug-bison@gnu.org
289X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
290
291The other day I had the need for explicitly building the parse tree. I
292used %locations for that and defined YYLLOC_DEFAULT to call a function
293that returns the tree node for the production. Easy. But I also needed
294to assign the S-attribute to the tree node. That cannot be done in
295YYLLOC_DEFAULT, because it is invoked before the action is executed.
296The way I solved this was to define a macro YYACT_EPILOGUE that would
297be invoked after the action. For reasons of symmetry I also added
298YYACT_PROLOGUE. Although I had no use for that I can envision how it
299might come in handy for debugging purposes.
76551463 300All is needed is to add
69991a58
AD
301
302#if YYLSP_NEEDED
303 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
304#else
305 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
306#endif
307
308at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
309
310I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
311to bison. If you're interested, I'll work on a patch.
312
35fe0834
PE
313* Better graphics
314Equip the parser with a means to create the (visual) parse tree.
d7215705 315
ff1b7a13
AD
316* Complaint submessage indentation.
317We already have an implementation that works fairly well for named
318reference messages, but it would be nice to use it consistently for all
319submessages from Bison. For example, the "previous definition"
320submessage or the list of correct values for a %define variable might
321look better with indentation.
322
323However, the current implementation makes the assumption that the
324location printed on the first line is not usually much shorter than the
325locations printed on the submessage lines that follow. That assumption
326may not hold true as often for some kinds of submessages especially if
327we ever support multiple grammar files.
328
329Here's a proposal for how a new implementation might look:
330
331 http://lists.gnu.org/archive/html/bison-patches/2009-09/msg00086.html
332
333
334Local Variables:
335mode: outline
336coding: utf-8
337End:
338
f294a2c2
AD
339-----
340
c932d613 341Copyright (C) 2001-2004, 2006, 2008-2012 Free Software Foundation, Inc.
f294a2c2 342
51cbef6f 343This file is part of Bison, the GNU Compiler Compiler.
f294a2c2 344
f16b0819 345This program is free software: you can redistribute it and/or modify
f294a2c2 346it under the terms of the GNU General Public License as published by
f16b0819
PE
347the Free Software Foundation, either version 3 of the License, or
348(at your option) any later version.
f294a2c2 349
f16b0819 350This program is distributed in the hope that it will be useful,
f294a2c2
AD
351but WITHOUT ANY WARRANTY; without even the implied warranty of
352MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
353GNU General Public License for more details.
354
355You should have received a copy of the GNU General Public License
f16b0819 356along with this program. If not, see <http://www.gnu.org/licenses/>.