Re: [9front] Patch to fix test(1) expression parsing

[ori@eigenstate.org]

> Hi all,
> 
> The following patch fixes test(1) to correctly parse its own
> expression syntax whilst avoiding executing of filesystem operations
> when these tests ought not to be performed due to short-circuiting.
> 
> Previously, short-circuiting of -a and -o logical operators would
> result in an incorrect parse.  For example, the following invocations

Generally looks good, will test tomorrow -- just a minor comment:

> fail:
> 
> term% test '(' 1 -gt 2 -a 1 -gt 1 ')'
> test: ) expected
> term% test '(' 2 -gt 1 -o 1 -gt 1 ')'
> test: ) expected
> 

> -	r = e();
> +	r = e(1);
>  	/*
>  	 * nice idea but short-circuit -o and -a operators may have
>  	 * not consumed their right-hand sides.

This comment seems outdated -- we can use the nice idea and
error out appropriately now.