Regression in Combinator Parsing library: parser generator rep1 doesn't propagate errors

[scabug]

The rewrite of rep1 in the Parsers trait in r20573 caused a regression of #SI-1100.

The inner method of rep1 should probably be changed to:

@tailrec def applyp(in0: Input): ParseResult[List[T]] = p0(in0) match {
  case Success(x, rest) => elems += x ; applyp(rest)
  case e: Error         => e
  case _                => Success(elems.toList, in0)
}

Just adding the line with case e: Error

(Is it ok to create a new issue in such cases, or better to reopen the old one?)

[scabug]

Imported From: https://issues.scala-lang.org/browse/SI-5108?orig=1
Reporter: Maarten van der Zwaart (mzwaart)
Affected Versions: 2.8.0, 2.9.2

[scabug]

Commit Message Bot (anonymous) said:
(extempore in r25881) Fix for combinator regression.

Propagate Error in repN. I have no time for a test case, I will gladly
take a contribution. References #1100, Closes #5108, No review.

[scabug]

@dcsobral said:
Pull request for test case: scala/scala#108

[scabug]

Commit Message Bot (anonymous) said:
(extempore in r25889) Test case for #1100/#5108.