jison doesn't handle indirect left recursion properly

[GerHobbelt]

Don't know yet if this is particular to my fork or the original already.

Ran into this issue when working on lex-parser: commits follow

[GerHobbelt]

• GerHobbelt/lex-parser@ceecbcb81cfe9eb6006ac5f06e8eee32832c6a2a temporarily revert to the quickfix in SHA-1: GerHobbelt/lex-parser@28a0e3498258cc0c13709c8ef7736b59fe3ce8c4 as it turns out GerHobbelt/jison#1 is a real problem
• GerHobbelt/lex-parser@f49e214fdf28bc8fcab1f9e8b9805b2fad2a8351: added failing test: fails when we have a grammar with indirect recursion A->B->A like we have now.
• GerHobbelt/lex-parser@1246dbb75472cee8e4e91318cc5a0d4739a8fe12: yup! indirect left-recursive equivalent is TOAST: cannot parse bnf.l (line 13, BR \r\n|\n|\r)
• GerHobbelt/lex-parser@28a0e3498258cc0c13709c8ef7736b59fe3ce8c4: hmmmm.... looks like jison has a problem with indirect left recursion... this grammar works and AFAICT its synonymous to the one in SHA GerHobbelt/lex-parser@cecdd0462d5e3b4c92d2ca764e79f57d1a78df5e
• GerHobbelt/lex-parser@b946affa9b7d3a3f16f7946867d9df120616adff: investigating why grammar fails to parse regexes all of a sudden; cannot see the error in my grammar :-(

[GerHobbelt]

Whoops. Don't understand I could have overlooked these! 👎 👎

Conflict in grammar: multiple actions possible when lookahead token is NAME in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is START_INC in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is START_EXC in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is { in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is ACTION in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is INCLUDE in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is OPTIONS in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is UNKNOWN_DECL in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is %% in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)
Conflict in grammar: multiple actions possible when lookahead token is | in state 18
- reduce by rule: regex_list -> nonempty_regex_list
- reduce by rule: regex -> nonempty_regex_list
  (Resolved R/R conflict: use first production declared in grammar.)