FMSLogo Code

Add EVALTEST.DEFINEAFTERRUN.
This tests that calling a procedure that isn't defined throws an "I don't know how to" error,
but that after the procedure is defined, the calling procedure is retreeified and runs correctly.

Move the declaration of body_ptr to where it's first assigned in untreeify_body.
This makes the code easier to read.

Add EVALTEST.REDEFINEWHILERUNNING, a regression test for Bug #309.

Fix Bug #606; Running a procedure with parse error no longer leaks memory.

Restore the printing of leaked node's contents by setting the 'NODE' magic number.
I had broken this by adding memory filling for uninitialized memory.

Fix Bug #608; SETLABELFONT no longer leaks memory when given an unknown font.
This was a regression introduced in version 8.3.0.

Note that Bug #404 was fixed in version 7.0.0.

This was unknowingly fixed by r4179 when ranged_integer_arg() was updated do to nothing if stopping_node==THROWING before it was called, as is the case when SETLABELFONT encounters a malformed element in the font vector input.

Fix a misleading comment.
The comment suggested that control flow probably reaches a particular line
for a macroized primitive. In fact, that line is reached for any macro,
primitive or not.

Rework the fix for Bug #602 so that it copies the argument list only in APPLY
and only if it's to be given to a primitive.

I thought that was what I had done in my original fix, but I was mislead
by the label name and comments. The place that the list was being copied
apply every time a primitive was called, not just when it was run by APPLY.

This change should result in faster execution relative to the previous fix,
although the improvement may be too small to be measurable.

Revert the fix for Bug #363.
This was really a work-around for Bug #505, which is has been fixed.
Therefore the work-around is no longer needed.