FMSLogo Code

Add tests for LPUT involving :ERRACT to test that bad input errors are recoverable.

Note that Bug #400 was fixed in FMSLogo 8.5.0.
I had fixed this as part of implementing Feature Request #136, having forgotten that there was a separate bug for this.

Extend EVALTEST.PARSEERRORAFTEROUTPUT to include when "()" is given as an input to a procedure.

Fix "FORWARD 100 5*" to run FORWARD 100 before throwing a "not enough inputs" error.
This is part of fixing Bug #250.

This is not 100% correct, as the infix expressions prior to the error won't
be evaluated in something like "1*2*3+4+". However, I don't this can have a
visible impact, as the built-in infix operators don't have side effects and
redefining them turns them into prefix operator.

Fix a typo in a comment

Fix "FORWARD 100 )" to run FORWARD 100 before throwing a "unexpected ')'" error.
This is part of fixing Bug #250.

This has an unexpected consequence of making "FORWARD)" throw a "not enough inputs to FORWARD"
error instead of a "unexpected ')'" error, as it used to. I don't think this is bad because
both errors apply and technically FORWARD was missing an input before the ")" is reached.

This latter change is why EVALTEST.UNEXPECTEDCLOSEPAREN was modified to move the right paren.
However, I did test that the new definition still causes an infinite loop in FMSLogo 8.4.0.

Refactoring: Rewrite llput to use CAppendableList.
This is shorter and clearer.

Cleanup: Remove an unnecessary special case in lcount() for NIL.
This is already handled by the is_list() case.
I suspect that, at one time, is_list didn't check for NULL before looking at the node type.

Always compile debugutils.cpp, but remove code if DEBUG isn't defined.
This saves the effort of figuring out how to include it when it's needed.

Fix "FD 100 5**6" to run FD 100 before throwing a "not enough inputs to *" error.
This is part of fixing Bug #250.