diff options
author | Akim Demaille <akim.demaille@gmail.com> | 2020-11-08 17:50:46 +0100 |
---|---|---|
committer | Akim Demaille <akim.demaille@gmail.com> | 2020-11-10 07:08:03 +0100 |
commit | a38d0b914507d2c9163eef84240bf1a059a0659f (patch) | |
tree | 1f3144fb95578c93107e8fc09553d7514f2ed8d5 /TODO | |
parent | 4b0cd01fb7a1586e6886207376c587516c3152b5 (diff) | |
download | bison-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-- | TODO | 19 |
1 files changed, 0 insertions, 19 deletions
@@ -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. |