Future/bison
1
0
Fork 0
mirror of https://git.savannah.gnu.org/git/bison.git synced 2026-03-18 16:53:02 +00:00
Code Issues Packages Projects Releases Wiki Activity

todo: more

Browse Source
This commit is contained in:
Akim Demaille
2020-06-23 07:33:03 +02:00
parent 01af4ad9c3
commit a0b044186b
1 changed files with 19 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
TODO
View File

@@ -672,6 +672,25 @@ participate in the count.
Do we want to disallow terminal start symbols? The limitation is not Do we want to disallow terminal start symbols? The limitation is not
technical. Can it be useful to someone to "parse" a token? technical. Can it be useful to someone to "parse" a token?
*** Fix default_reduction_only_for_accept
In src/lalr.c:
/* We need a lookahead either to distinguish different reductions
(i.e., there are two or more), or to distinguish a reduction from a
shift. Otherwise, it is straightforward, and the state is
'consistent'. However, do not treat a state with any reductions as
consistent unless it is the accepting state (because there is never
a lookahead token that makes sense there, and so no lookahead token
should be read) if the user has otherwise disabled default
reductions. */
s->consistent =
!(reds->num > 1
|| (reds->num == 1 && trans->num && TRANSITION_IS_SHIFT (trans, 0))
|| (reds->num == 1 && reds->rules[0]->number != 0
&& default_reduction_only_for_accept));
We should have a test for "is accepting rule".
** %include ** %include
This is a popular demand. We already made many changes in the parser that This is a popular demand. We already made many changes in the parser that
should make this reasonably easy to implement. should make this reasonably easy to implement.