summaryrefslogtreecommitdiff
path: root/TODO
diff options
context:
space:
mode:
authorAkim Demaille <akim.demaille@gmail.com>2020-11-08 17:50:46 +0100
committerAkim Demaille <akim.demaille@gmail.com>2020-11-10 07:08:03 +0100
commita38d0b914507d2c9163eef84240bf1a059a0659f (patch)
tree1f3144fb95578c93107e8fc09553d7514f2ed8d5 /TODO
parent4b0cd01fb7a1586e6886207376c587516c3152b5 (diff)
downloadbison-a38d0b914507d2c9163eef84240bf1a059a0659f.tar.gz
multistart: introduce and use rule_is_initial
* src/gram.h (rule_is_initial): New. * src/graphviz.c, src/print-xml.c, src/print.c, src/lalr.c: Use it. Some of these occurrences were incorrect (checking whether this is rule 0), and not behaving properly in the case of multistart.
Diffstat (limited to 'TODO')
-rw-r--r--TODO19
1 files changed, 0 insertions, 19 deletions
diff --git a/TODO b/TODO
index 8b009d08..c6a71a34 100644
--- a/TODO
+++ b/TODO
@@ -685,25 +685,6 @@ participate in the count.
Do we want to disallow terminal start symbols? The limitation is not
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
This is a popular demand. We already made many changes in the parser that
should make this reasonably easy to implement.