FMSLogo Code

Fix two simple typos: a misspelling of "is" and an extra newline before the code sample.

Fix bug #1543354: REVERSE now throws an error when given two inputs.

The fix was to provide a wrapper that takes only one input and calls a helper function (the original REVERSE) which takes two inputs.

Add a function comment.

Add a regression test for bug #1543643: which is that FMSLogo becomes unresponsive when printing a very long string to the commander's recall box.

Fix bug #1543643: FMSLogo is responsive when running 'PR ISEQ 1 100000'. This is the first bugfix for FMSLogo 6.11.0.

This fix involved making two optimizations, rather than just scanning for windows messages more often:
1) When we reallocate combo_buff, we always at least double its size. This avoids a problem where we thrash the allocator by always reallocating the buffer to be one byte larger than the previous size. This changes the use of allocation to O(N) to O(log N).
2) Remove many calls to strlen(combo_buff) by adding combo_buff_length, and keeping it up-to-date. This changes the printing routine from being O(N^2) to O(N).

Document that REVERSEworks on words, as well as lists. Also document that it does NOT work on arrays.

Refactor llput() a little:
* Choose better names for stack variables.
* Add multiple return points for when the input is bad.

Fix a false-postive.

Remove the logging for the third and fourth fields of ERROR. These were generally not informative and added a lot of noise to the test log. In the few cases where these values should be checked, they should be explicitly checked by the caller.

Add regression tests for LPUT, FPUT, and REVERSE.