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