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@ceecbcb temporarily revert to the quickfix in SHA-1: GerHobbelt/lex-parser@28a0e34 as it turns out jison doesn't handle indirect left recursion properly #1 is a real problem
• GerHobbelt/lex-parser@f49e214: added failing test: fails when we have a grammar with indirect recursion A->B->A like we have now.
• GerHobbelt/lex-parser@1246dbb: yup! indirect left-recursive equivalent is TOAST: cannot parse bnf.l (line 13, BR \r\n|\n|\r)
• GerHobbelt/lex-parser@28a0e34: 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@cecdd04
• GerHobbelt/lex-parser@b946aff: 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.)