FMSLogo Code

Add more tests for NUMBERP to prepare for some refactoring in that procedure.

Cleanup: Remove a work-around for a bug in ARRAYTOLIST that has now been fixed.

Add an optimization to LOGGABLEWORD that returns the given word.
In the common case, the words do not need to be escaped.

Add tests for LOGGABLEWORD

Refactoring: Use FAILIFNOTFALSE instead of duplicating its logic.

Simplify the implementation of is_barred into a single boolean expression.

Refactoring: Simplify a ref() and a deref() into a gcref().

Update expected results of test pass to include recent bug fixes and new tests cases.

Fix a memory leak when parsing some numbers.
This leak was introduced when I fixed cnv_node_to_numnode not to free its argument when it was a string that had zero references.
Even though it made these callers more complex, I still think the change to cnv_node_to_numnode is correct.
The code base is simpler if cvn_node_to_numnode never frees its argument.

Fix a freed memory read caused by cnv_node_to_numnode when its argument has zero references.
This bug has always existed but wasn't exposed until I recently started to remove side effects from its callers.
One of the side effects which I removed removed ensured that the argument never had zero references.
I still think that removing side effects is benefial to the code base, even if it removes some hidden assumptions.