FMSLogo Code

Refactoring: Rework lnumberp to use numberp instead of cnv_node_to_numnode.
This is simpler and avoids the side effect of modifying args.
This is part of a larger effort to reduce the number of side effects in the code base.

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.