FMSLogo Code

Refactoring: inline the last occurrence of integer_arg.
This is part of a larger effort to reduce the number of invisible side effects in the code base.

Refactoring: Replace the last remaining list_arg call with a call to validate_list_argument.
validate_list_argument doesn't modify the arguments list.
This is part of a larger effort to reduce the number of side effects in the code base.

Add test coverage that giving bad input to SELECTBOX is a recoverable error.
The tests were adapted from lput.lgo.
At this point, the tests require some manual intervention, but they will be automated later.

Refactoring: Rework lfput() so that it doesn't modify its arguments list.
This is part of a larger effort to reduce the number of side effects in the code base.

Add test coverage that giving bad input to FPUT is a recoverable error.
The tests were adapted from lput.lgo.

Refactoring: Change llput to not modify its arguments list.
This is part of a larger effort to reduce the number of side effects in the code base.

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.