Question about behavior of repetitions

[MarcelBolten]

Hi all,

I think there is a discrepancy between the documentation for repetitions in conjunction with code blocks and results of preceding labeled expressions and the actual behavior.
In particular the statement

Any non-number values, returned by the code block, will be interpreted as 0.

seems to be wrong.
A grammar like:

start = text:"a"|{ return 'some string' }|

does not return a number and hence should be equivalent to

start = text:"a"|0|

However, the parser accepts infinite input of "a".

This is probably due to the following lines, in particular |0 and the operator precedence?:

peggy/lib/compiler/passes/generate-js.js

Lines 490 to 496 in d2f5bb9

	case op.IF_LT_DYNAMIC: // IF_LT_DYNAMIC min, t, f
	compileCondition(stack.top() + ".length < " + stack.index(bc[ip + 1]) + "|0", 1);
	break;
	case op.IF_GE_DYNAMIC: // IF_GE_DYNAMIC max, t, f
	compileCondition(stack.top() + ".length >= " + stack.index(bc[ip + 1]) + "|0", 1);
	break;

Any clarification is appreciated.

[Mingun]

Yes, this is a bug. Need to wrap boundary|0 in parenthesis