]> git.saurik.com Git - bison.git/blame - TODO
YYERRCODE.
[bison.git] / TODO
CommitLineData
416bd7a9
MA
1-*- outline -*-
2
42f832d6 3* Various
865f1e9f
AD
4** YYPRINT
5glr.c inherits its symbol_print function from c.m4, which supports
6YYPRINT. But to use YYPRINT yytoknum is needed, which not defined by
7glr.c.
8
9Anyway, IMHO YYPRINT is obsolete and should be restricted to yacc.c.
10
42f832d6
AD
11** YYERRCODE
12Defined to 256, but not used, not documented. Probably the token
13number for the error token, which POSIX wants to be 256, but which
14Bison might renumber if the user used number 256. Keep fix and doc?
15Throw away?
16
fc2476c7
AD
17We could (should?) also treat the case of the undef_token, which is
18numbered 257 for yylex, and 2 internal. Both appear for instance in
19toknum:
20
21 const unsigned short int
22 parser::yytoken_number_[] =
23 {
24 0, 256, 257, 258, 259, 260, 261, 262, 263, 264,
25
26while here
27
28 enum yytokentype {
29 TOK_EOF = 0,
30 TOK_EQ = 258,
31
32so both 256 and 257 are "mysterious".
33
34 const char*
35 const parser::yytname_[] =
36 {
37 "\"end of command\"", "error", "$undefined", "\"=\"", "\"break\"",
38
39
42f832d6
AD
40** YYFAIL
41It is seems to be *really* obsolete now, shall we remove it?
42
43** YYBACKUP
44There is no test about it, no examples in the doc, and I'm not sure
45what it should look like. For instance what follows crashes.
46
47 %error-verbose
48 %debug
49 %pure-parser
50 %code {
51 # include <stdio.h>
52 # include <stdlib.h>
53 # include <assert.h>
54
55 static void yyerror (const char *msg);
56 static int yylex (YYSTYPE *yylval);
57 }
58 %%
59 exp:
60 'a' { printf ("a: %d\n", $1); }
61 | 'b' { YYBACKUP('a', 123); }
62 ;
63 %%
64 static int
65 yylex (YYSTYPE *yylval)
66 {
67 static char const input[] = "b";
68 static size_t toknum;
69 assert (toknum < sizeof input);
70 *yylval = (toknum + 1) * 10;
71 return input[toknum++];
72 }
73
74 static void
75 yyerror (const char *msg)
76 {
77 fprintf (stderr, "%s\n", msg);
78 }
79
80 int
81 main (void)
82 {
83 yydebug = !!getenv("YYDEBUG");
84 return yyparse ();
85 }
86
27cb5b59
AD
87** yychar == yyempty_
88The code in yyerrlab reads:
89
90 if (yychar <= YYEOF)
91 {
92 /* Return failure if at end of input. */
93 if (yychar == YYEOF)
94 YYABORT;
95 }
96
97There are only two yychar that can be <= YYEOF: YYEMPTY and YYEOF.
98But I can't produce the situation where yychar is YYEMPTY here, is it
99really possible? The test suite does not exercise this case.
100
101This shows that it would be interesting to manage to install skeleton
102coverage analysis to the test suite.
42f832d6 103
a2e3fa77
AD
104** Table definitions
105It should be very easy to factor the definition of the various tables,
106including the separation bw declaration and definition. See for
107instance b4_table_define in lalr1.cc. This way, we could even factor
108C vs. C++ definitions.
109
00a8a083
AD
110* From lalr1.cc to yacc.c
111** Single stack
112Merging the three stacks in lalr1.cc simplified the code, prompted for
113other improvements and also made it faster (probably because memory
114management is performed once instead of three times). I suggest that
115we do the same in yacc.c.
116
117** yysyntax_error
118In lalr1.cc we invoke it with the translated lookahead (yytoken), and
119yacc.c uses yychar. I don't see why.
120
dada3cd1
AD
121** yysyntax_error
122The use of switch to select yyfmt in lalr1.cc seems simpler than
123what's done in yacc.c.
124
3c146b5e
AD
125* Header guards
126
32f0598d 127From Franc,ois: should we keep the directory part in the CPP guard?
3c146b5e
AD
128
129
c19988b7
AD
130* Yacc.c: CPP Macros
131
132Do some people use YYPURE, YYLSP_NEEDED like we do in the test suite?
133They should not: it is not documented. But if they need to, let's
134find something clean (not like YYLSP_NEEDED...).
135
136
5d278082
PE
137* Installation
138
88bce5a2 139* Documentation
959e5f51
AD
140Before releasing, make sure the documentation ("Understanding your
141parser") refers to the current `output' format.
88bce5a2 142
2ab9a04f 143* Report
ec3bc396 144
2ab9a04f
AD
145** GLR
146How would Paul like to display the conflicted actions? In particular,
742e4900 147what when two reductions are possible on a given lookahead token, but one is
2ab9a04f
AD
148part of $default. Should we make the two reductions explicit, or just
149keep $default? See the following point.
d7215705 150
2ab9a04f
AD
151** Disabled Reductions
152See `tests/conflicts.at (Defaulted Conflicted Reduction)', and decide
153what we want to do.
d7215705 154
2ab9a04f 155** Documentation
bc933ef1
AD
156Extend with error productions. The hard part will probably be finding
157the right rule so that a single state does not exhibit too many yet
158undocumented ``features''. Maybe an empty action ought to be
159presented too. Shall we try to make a single grammar with all these
160features, or should we have several very small grammars?
ec3bc396 161
2ab9a04f
AD
162** --report=conflict-path
163Provide better assistance for understanding the conflicts by providing
164a sample text exhibiting the (LALR) ambiguity. See the paper from
165DeRemer and Penello: they already provide the algorithm.
166
38eb7751
PE
167** Statically check for potential ambiguities in GLR grammars. See
168<http://www.i3s.unice.fr/~schmitz/papers.html#expamb> for an approach.
169
ec3bc396 170
948be909 171* Extensions
2ab9a04f 172
d2aaf69e 173** Labeling the symbols
959e5f51
AD
174Have a look at the Lemon parser generator: instead of $1, $2 etc. they
175can name the values. This is much more pleasant. For instance:
176
177 exp (res): exp (a) '+' exp (b) { $res = $a + $b; };
178
179I love this. I have been bitten too often by the removal of the
180symbol, and forgetting to shift all the $n to $n-1. If you are
181unlucky, it compiles...
182
d2aaf69e
AD
183But instead of using $a etc., we can use regular variables. And
184instead of using (), I propose to use `:' (again). Paul suggests
185supporting `->' in addition to `:' to separate LHS and RHS. In other
186words:
187
188 r:exp -> a:exp '+' b:exp { r = a + b; };
189
190That requires an significant improvement of the grammar parser. Using
191GLR would be nice. It also requires that Bison know the type of the
192symbols (which will be useful for %include anyway). So we have some
193time before...
194
195Note that there remains the problem of locations: `@r'?
196
197
959e5f51
AD
198** $-1
199We should find a means to provide an access to values deep in the
200stack. For instance, instead of
201
202 baz: qux { $$ = $<foo>-1 + $<bar>0 + $1; }
203
204we should be able to have:
205
206 foo($foo) bar($bar) baz($bar): qux($qux) { $baz = $foo + $bar + $qux; }
207
208Or something like this.
209
f0e48240
AD
210** %if and the like
211It should be possible to have %if/%else/%endif. The implementation is
212not clear: should it be lexical or syntactic. Vadim Maslow thinks it
213must be in the scanner: we must not parse what is in a switched off
214part of %if. Akim Demaille thinks it should be in the parser, so as
215to avoid falling into another CPP mistake.
216
ca752c34
AD
217** XML Output
218There are couple of available extensions of Bison targeting some XML
219output. Some day we should consider including them. One issue is
220that they seem to be quite orthogonal to the parsing technique, and
221seem to depend mostly on the possibility to have some code triggered
222for each reduction. As a matter of fact, such hooks could also be
223used to generate the yydebug traces. Some generic scheme probably
224exists in there.
225
226XML output for GNU Bison and gcc
227 http://www.cs.may.ie/~jpower/Research/bisonXML/
228
229XML output for GNU Bison
230 http://yaxx.sourceforge.net/
f0e48240 231
fa770c86
AD
232* Unit rules
233Maybe we could expand unit rules, i.e., transform
234
235 exp: arith | bool;
236 arith: exp '+' exp;
237 bool: exp '&' exp;
238
239into
240
241 exp: exp '+' exp | exp '&' exp;
242
243when there are no actions. This can significantly speed up some
d7215705
AD
244grammars. I can't find the papers. In particular the book `LR
245parsing: Theory and Practice' is impossible to find, but according to
246`Parsing Techniques: a Practical Guide', it includes information about
247this issue. Does anybody have it?
fa770c86 248
51dec47b 249
51dec47b 250
2ab9a04f 251* Documentation
51dec47b 252
2ab9a04f
AD
253** History/Bibliography
254Some history of Bison and some bibliography would be most welcome.
255Are there any Texinfo standards for bibliography?
256
257
258
948be909
PE
259* Java, Fortran, etc.
260
948be909 261
2ab9a04f
AD
262* Coding system independence
263Paul notes:
264
265 Currently Bison assumes 8-bit bytes (i.e. that UCHAR_MAX is
266 255). It also assumes that the 8-bit character encoding is
267 the same for the invocation of 'bison' as it is for the
268 invocation of 'cc', but this is not necessarily true when
269 people run bison on an ASCII host and then use cc on an EBCDIC
270 host. I don't think these topics are worth our time
271 addressing (unless we find a gung-ho volunteer for EBCDIC or
272 PDP-10 ports :-) but they should probably be documented
273 somewhere.
fa770c86 274
d521d95a
PE
275 More importantly, Bison does not currently allow NUL bytes in
276 tokens, either via escapes (e.g., "x\0y") or via a NUL byte in
277 the source code. This should get fixed.
aef1ffd5 278
bcb05e75 279* --graph
45567173 280Show reductions.
bcb05e75 281
704a47c4 282* Broken options ?
45567173
AD
283** %token-table
284** Skeleton strategy
728c4be2 285Must we keep %token-table?
416bd7a9 286
0e95c1dd 287* BTYacc
f0e48240 288See if we can integrate backtracking in Bison. Charles-Henri de
df72984a
AD
289Boysson <de-boy_c@epita.fr> has been working on this, but never gave
290the results.
291
292Vadim Maslow, the maintainer of BTYacc was once contacted. Adjusting
293the Bison grammar parser will be needed to support some extra BTYacc
294features. This is less urgent.
0e95c1dd 295
2ab9a04f
AD
296** Keeping the conflicted actions
297First, analyze the differences between byacc and btyacc (I'm referring
298to the executables). Find where the conflicts are preserved.
299
300** Compare with the GLR tables
948be909 301See how isomorphic the way BTYacc and the way the GLR adjustments in
2ab9a04f
AD
302Bison are compatible. *As much as possible* one should try to use the
303same implementation in the Bison executables. I insist: it should be
304very feasible to use the very same conflict tables.
305
306** Adjust the skeletons
307Import the skeletons for C and C++.
308
0e95c1dd
AD
309
310* Precedence
2ab9a04f
AD
311
312** Partial order
0e95c1dd
AD
313It is unfortunate that there is a total order for precedence. It
314makes it impossible to have modular precedence information. We should
2ab9a04f 315move to partial orders (sounds like series/parallel orders to me).
0e95c1dd 316
2ab9a04f
AD
317** RR conflicts
318See if we can use precedence between rules to solve RR conflicts. See
319what POSIX says.
320
321
69991a58
AD
322* $undefined
323From Hans:
324- If the Bison generated parser experiences an undefined number in the
325character range, that character is written out in diagnostic messages, an
326addition to the $undefined value.
327
328Suggest: Change the name $undefined to undefined; looks better in outputs.
329
2ab9a04f 330
69991a58
AD
331* Default Action
332From Hans:
333- For use with my C++ parser, I transported the "switch (yyn)" statement
334that Bison writes to the bison.simple skeleton file. This way, I can remove
335the current default rule $$ = $1 implementation, which causes a double
336assignment to $$ which may not be OK under C++, replacing it with a
337"default:" part within the switch statement.
338
339Note that the default rule $$ = $1, when typed, is perfectly OK under C,
340but in the C++ implementation I made, this rule is different from
341$<type_name>$ = $<type_name>1. I therefore think that one should implement
342a Bison option where every typed default rule is explicitly written out
343(same typed ruled can of course be grouped together).
344
345* Pre and post actions.
346From: Florian Krohm <florian@edamail.fishkill.ibm.com>
347Subject: YYACT_EPILOGUE
348To: bug-bison@gnu.org
349X-Sent: 1 week, 4 days, 14 hours, 38 minutes, 11 seconds ago
350
351The other day I had the need for explicitly building the parse tree. I
352used %locations for that and defined YYLLOC_DEFAULT to call a function
353that returns the tree node for the production. Easy. But I also needed
354to assign the S-attribute to the tree node. That cannot be done in
355YYLLOC_DEFAULT, because it is invoked before the action is executed.
356The way I solved this was to define a macro YYACT_EPILOGUE that would
357be invoked after the action. For reasons of symmetry I also added
358YYACT_PROLOGUE. Although I had no use for that I can envision how it
359might come in handy for debugging purposes.
76551463 360All is needed is to add
69991a58
AD
361
362#if YYLSP_NEEDED
363 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen, yyloc, (yylsp - yylen));
364#else
365 YYACT_EPILOGUE (yyval, (yyvsp - yylen), yylen);
366#endif
367
368at the proper place to bison.simple. Ditto for YYACT_PROLOGUE.
369
370I was wondering what you think about adding YYACT_PROLOGUE/EPILOGUE
371to bison. If you're interested, I'll work on a patch.
372
35fe0834
PE
373* Better graphics
374Equip the parser with a means to create the (visual) parse tree.
d7215705 375
f294a2c2
AD
376-----
377
df72984a 378Copyright (C) 2001, 2002, 2003, 2004, 2006, 2008 Free Software Foundation,
51cbef6f 379Inc.
f294a2c2 380
51cbef6f 381This file is part of Bison, the GNU Compiler Compiler.
f294a2c2 382
f16b0819 383This program is free software: you can redistribute it and/or modify
f294a2c2 384it under the terms of the GNU General Public License as published by
f16b0819
PE
385the Free Software Foundation, either version 3 of the License, or
386(at your option) any later version.
f294a2c2 387
f16b0819 388This program is distributed in the hope that it will be useful,
f294a2c2
AD
389but WITHOUT ANY WARRANTY; without even the implied warranty of
390MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
391GNU General Public License for more details.
392
393You should have received a copy of the GNU General Public License
f16b0819 394along with this program. If not, see <http://www.gnu.org/licenses/>.