FMSLogo Code

Fix a regression in is_freed() caused when NODETYPE was changed from a short to an enum.
The memset of 0xCC now changes type to 0xCCCCCCCC intsead of 0xCCCC.
This regression caused me not to notice a freed memory read that I had introduced in BITAND because it wasn't detected in debug builds.

Fix a typo in a comment

Automate the button clicks in SELECTBOXTEST.BADINPUT.RECOVERABLE.

Cleanup: Remove unused function integer_arg.

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.