FMSLogo Code

Fix bug #1535582: SELECTBOX now throws an error on bad choicelist input.

Also did some code cleanup on lquestionbox().

Cleanup the wording on some of the QUESTIONBOX tests.
Add a test for when the answer has a space in it.

Add a regression test for calling SELECTBOX with an empty choicelist input.

Extract conditional: Create a function that determines if a numeric NODE* is non-negative, rather that copying the condition inline in several places.

Also, rename the "floatok" parameter to "NegativeIsOk" because that's how it is used. I don't know why it was called floatok, but I suspect this was a vestige from long ago.

I also make the vector_4_arg() consistent with the other routines in allowing floating point numbers. This condition is currently unused, but if it does get used in the future, it'll be nice to have consistent behavior.

Add regression tests for PENSIZE and SETPENSIZE.

Add tests that give SETACTIVEAREA 0 and 0.0. This is a glass-box test because there is special code to handle this case.

Remove the reference from the parameter to GetRGBorIndexColor(). This was generating a compile-time warning with good reason, because a temporary was being passed in. This does not cause a problem, because the reference isn't modified, which means using a reference is unnecessary.
The use of a reference dates back to before FMSLogo, so I don't know why it's like that, but I suspect that it was a mistake.

This change also contains some minor formatting and cosmetic changes.

Refactoring: Rename member variables of TClientRectangle to begin with an "m_" to show their scope.

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.