Revert "OneOrMoreMatcher: skip test for empty match"

This reverts commit 35119aab5491906920df6b8b3ba5b5de781cbdff.

It appears that this test is done in each and every matcher _in addition_ to the
CanMatchEmptyVisitor.

There needs to be a way to detect that earlier so that such a test does not have
to be done at "parsing runtime"!