FMSLogo Code

Refactoring: Replace a for loop with no initialization statement with a while loop.
This makes the code structure clearer.

Refactoring: Move a local variable declaration and assignment closer to where the variable is first used.
This makes the code easier to read.

Update test log with output from new tests.

Fix Bug #600; A procedure which runs a tail-called APPLY no longer leaks its parameters into the caller's scope.

The fix was to split llocal() into two procedures, one that creates a variable scope and one that doesn't.
LOCAL uses the one that creates a new variable scope (no change).
evaluator() uses the one that doesn't (the fix).

This change diverges from UCBLogo's implementation, which also doesn't have this bug.
Therefore, there's at least one other way fix it. I'm not confident that I picked the right fix.

Add a bad input test to APPLY that crashes UCBLogo but not FMSLogo.
This is important, in case I cherry-pick changes from UCBLogo to fix problems in FMSLogo.

Rewrite some tests which log very long lines to instead log short lines.
This addresses a problem when using a side-by-side differ to compare the test log to the baseline test log.
Notably, meld has a limit of 8K characters per line.

Use wxSingleInstanceChecker to detect if another instance of FMSLogo is running on startup.
This prevents multiple instances of FMSLogo from running on GNU/Linux unless to they were given a specific program to run.
The CreateMutex logic on Windows was preserved until it can be confirmed that wxSingleInstanceChecker can grab the exact same mutex.
Otherwise, FMSLogo 8.5.0 couldn't detect if FMSLogo 8.4.0, the installer, or the uninstaller is running, and vice versa.

Refactoring: Rename NODETYPES to NODETYPE.
When the type was an "unsigned short", a plural made sense because it was the set of all possible types.
As an enum, a singular makes more sense, because it's the type of the associated NODE.

Change NODETYPES from "unsigned short" to an enum.
This adds a little more type safety and makes debugging easier (the debugger shows the symbolic name, not the numeric value).
While this increased the size of the type from 2 to 4 in gcc, it didn't increase the size of the NODE.
If the new size of NODETYPES proves to be problematic, it can be changed to an "enum : unsigned short".

Fix Bug #598; QUEUE and DEQUEUE now behave correctly when the queue name is shadowed by one of their own local variables.
The fix is to rewrite these procedures as macros that get evaluated in the caller's context.
For performance, when the queue name doesn't clash with a local variable, the previous implementation is run.