Odd behavioral intersection of combinators (alt/or and times).

I am experiencing behavior which I both don’t expect, and do not immediately understand. It seems like however times is implemented breaks the usage of the fallback combinators. Both alt and .or seem to break when used with times. Example:

const P = require('parsimmon');

P.digit.times(2).or(P.any.atLeast(1).tie()).tryParse('56a') // throws exception because the first parser fails.

P.alt(
  P.digit.times(2),
  P.any.atLeast(1).tie()
).tryParse('56a') // also fails

This would seem to be incorrect behavior. I can work on a fix if anyone can corroborate that this is not supposed to behave this way.

[theqabalist]

oh wait, if I make the EOF explicit, then it would actually fail. So something like seq(digits, EOF) would fail in the alt.

[wavebeem]

Based on https://github.com/jneen/parsimmon/issues/143 I’m likely to go ahead and implement https://github.com/jneen/parsimmon/issues/193 (which you could monkey patch locally and play with)