]> git.saurik.com Git - bison.git/blame - TODO
build: fix ChangeLog generation.
[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
c19988b7
AD
129* Yacc.c: CPP Macros
130
131Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite?
132They should not: it is not documented. But if they need to, let's
133find something clean (not like YYLSP_NEEDED...).
134
2ab9a04f 135* Report
ec3bc396 136
ff1b7a13
AD
137** Figures
138Some statistics about the grammar and the parser would be useful,
139especially when asking the user to send some information about the
140grammars she is working on. We should probably also include some
141information about the variables (I'm not sure for instance we even
142specify what LR variant was used).
143
2ab9a04f
AD
144** GLR
145How would Paul like to display the conflicted actions? In particular,
742e4900 146what when two reductions are possible on a given lookahead token, but one is
2ab9a04f
AD
147part of $default. Should we make the two reductions explicit, or just
148keep $default? See the following point.
d7215705 149
2ab9a04f
AD
150** Disabled Reductions
151See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
152what we want to do.
d7215705 153
2ab9a04f 154** Documentation
bc933ef1
AD
155Extend with error productions. The hard part will probably be finding
156the right rule so that a single state does not exhibit too many yet
157undocumented ``features''. Maybe an empty action ought to be
158presented too. Shall we try to make a single grammar with all these
159features, or should we have several very small grammars?
ec3bc396 160
2ab9a04f
AD
161** --report=conflict-path
162Provide better assistance for understanding the conflicts by providing
163a sample text exhibiting the (LALR) ambiguity. See the paper from
164DeRemer and Penello: they already provide the algorithm.
165
38eb7751
PE
166** Statically check for potential ambiguities in GLR grammars. See
167<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
168
ec3bc396 169
948be909 170* Extensions
2ab9a04f 171
959e5f51
AD
172** $-1
173We should find a means to provide an access to values deep in the
174stack. For instance, instead of
175
ff1b7a13 176 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
959e5f51
AD
177
178we should be able to have:
179
180 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
181
182Or something like this.
183
f0e48240
AD
184** %if and the like
185It should be possible to have %if/%else/%endif. The implementation is
186not clear: should it be lexical or syntactic. Vadim Maslow thinks it
187must be in the scanner: we must not parse what is in a switched off
188part of %if. Akim Demaille thinks it should be in the parser, so as
189to avoid falling into another CPP mistake.
190
ca752c34
AD
191** XML Output
192There are couple of available extensions of Bison targeting some XML
193output. Some day we should consider including them. One issue is
194that they seem to be quite orthogonal to the parsing technique, and
195seem to depend mostly on the possibility to have some code triggered
196for each reduction. As a matter of fact, such hooks could also be
197used to generate the yydebug traces. Some generic scheme probably
198exists in there.
199
200XML output for GNU Bison and gcc
201 http://www.cs.may.ie/~jpower/Research/bisonXML/
202
203XML output for GNU Bison
204 http://yaxx.sourceforge.net/
f0e48240 205
fa770c86
AD
206* Unit rules
207Maybe we could expand unit rules, i.e., transform
208
ff1b7a13
AD
209 exp: arith | bool;
210 arith: exp '+' exp;
211 bool: exp '&' exp;
fa770c86
AD
212
213into
214
ff1b7a13 215 exp: exp '+' exp | exp '&' exp;
fa770c86
AD
216
217when there are no actions. This can significantly speed up some
d7215705
AD
218grammars. I can't find the papers. In particular the book `LR
219parsing: Theory and Practice' is impossible to find, but according to
220`Parsing Techniques: a Practical Guide', it includes information about
221this issue. Does anybody have it?
fa770c86 222
51dec47b 223
51dec47b 224
2ab9a04f 225* Documentation
51dec47b 226
2ab9a04f
AD
227** History/Bibliography
228Some history of Bison and some bibliography would be most welcome.
229Are there any Texinfo standards for bibliography?
230
2ab9a04f
AD
231* Coding system independence
232Paul notes:
233
ff1b7a13
AD
234 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
235 255). It also assumes that the 8-bit character encoding is
236 the same for the invocation of 'bison' as it is for the
237 invocation of 'cc', but this is not necessarily true when
238 people run bison on an ASCII host and then use cc on an EBCDIC
239 host. I don't think these topics are worth our time
240 addressing (unless we find a gung-ho volunteer for EBCDIC or
241 PDP-10 ports :-) but they should probably be documented
242 somewhere.
fa770c86 243
ff1b7a13
AD
244 More importantly, Bison does not currently allow NUL bytes in
245 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
246 the source code. This should get fixed.
aef1ffd5 247
bcb05e75 248* --graph
45567173 249Show reductions.
bcb05e75 250
704a47c4 251* Broken options ?
45567173
AD
252** %token-table
253** Skeleton strategy
728c4be2 254Must we keep %token-table?
416bd7a9 255
0e95c1dd 256* Precedence
2ab9a04f
AD
257
258** Partial order
0e95c1dd
AD
259It is unfortunate that there is a total order for precedence. It
260makes it impossible to have modular precedence information. We should
2ab9a04f 261move to partial orders (sounds like series/parallel orders to me).
0e95c1dd 262
2ab9a04f
AD
263** RR conflicts
264See if we can use precedence between rules to solve RR conflicts. See
265what POSIX says.
266
267
69991a58
AD
268* $undefined
269From Hans:
270- If the Bison generated parser experiences an undefined number in the
271character range, that character is written out in diagnostic messages, an
272addition to the $undefined value.
273
274Suggest: Change the name $undefined to undefined; looks better in outputs.
275
2ab9a04f 276
69991a58
AD
277* Default Action
278From Hans:
279- For use with my C++ parser, I transported the "switch (yyn)" statement
280that Bison writes to the bison.simple skeleton file. This way, I can remove
281the current default rule $$ = $1 implementation, which causes a double
282assignment to $$ which may not be OK under C++, replacing it with a
283"default:" part within the switch statement.
284
285Note that the default rule $$ = $1, when typed, is perfectly OK under C,
286but in the C++ implementation I made, this rule is different from
287$<type_name>$ = $<type_name>1. I therefore think that one should implement
288a Bison option where every typed default rule is explicitly written out
289(same typed ruled can of course be grouped together).
290
291* Pre and post actions.
292From: Florian Krohm <florian@edamail.fishkill.ibm.com>
293Subject: YYACT_EPILOGUE
294To: bug-bison@gnu.org
295X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
296
297The other day I had the need for explicitly building the parse tree. I
298used %locations for that and defined YYLLOC_DEFAULT to call a function
299that returns the tree node for the production. Easy. But I also needed
300to assign the S-attribute to the tree node. That cannot be done in
301YYLLOC_DEFAULT, because it is invoked before the action is executed.
302The way I solved this was to define a macro YYACT_EPILOGUE that would
303be invoked after the action. For reasons of symmetry I also added
304YYACT_PROLOGUE. Although I had no use for that I can envision how it
305might come in handy for debugging purposes.
76551463 306All is needed is to add
69991a58
AD
307
308#if YYLSP_NEEDED
309 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
310#else
311 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
312#endif
313
314at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
315
316I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
317to bison. If you're interested, I'll work on a patch.
318
35fe0834
PE
319* Better graphics
320Equip the parser with a means to create the (visual) parse tree.
d7215705 321
ff1b7a13
AD
322* Complaint submessage indentation.
323We already have an implementation that works fairly well for named
324reference messages, but it would be nice to use it consistently for all
325submessages from Bison. For example, the "previous definition"
326submessage or the list of correct values for a %define variable might
327look better with indentation.
328
329However, the current implementation makes the assumption that the
330location printed on the first line is not usually much shorter than the
331locations printed on the submessage lines that follow. That assumption
332may not hold true as often for some kinds of submessages especially if
333we ever support multiple grammar files.
334
335Here's a proposal for how a new implementation might look:
336
337 http://lists.gnu.org/archive/html/bison-patches/2009-09/msg00086.html
338
339
340Local Variables:
341mode: outline
342coding: utf-8
343End:
344
f294a2c2
AD
345-----
346
c932d613 347Copyright (C) 2001-2004, 2006, 2008-2012 Free Software Foundation, Inc.
f294a2c2 348
51cbef6f 349This file is part of Bison, the GNU Compiler Compiler.
f294a2c2 350
f16b0819 351This program is free software: you can redistribute it and/or modify
f294a2c2 352it under the terms of the GNU General Public License as published by
f16b0819
PE
353the Free Software Foundation, either version 3 of the License, or
354(at your option) any later version.
f294a2c2 355
f16b0819 356This program is distributed in the hope that it will be useful,
f294a2c2
AD
357but WITHOUT ANY WARRANTY; without even the implied warranty of
358MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
359GNU General Public License for more details.
360
361You should have received a copy of the GNU General Public License
f16b0819 362along with this program. If not, see <http://www.gnu.org/licenses/>.