FMSLogo Code

Add a comment that eval.lgo also tests the treeification process in paren.cpp.

Fix a comment

Fix a typo in a comment

Add LOADTEST.REDEFINEWHILETREEIFYING, a regression test for bug #249.

Bug #249 is a memory leak when treeiying a procedure (which happens when it's first run) causes a file to load that redefines that procedure.

This is similar to EVALTEST.SILENTLOADFROMTREEIFY, except that this test is designed only to test the memory leak.

Add assert(!is_freed(nd)) to nodetype().
This required making is_freed return false when given NIL.
This was done to make freed memory use bugs easier to find.

Fix a memory leak in LPUT when it had bad input and the :erract handler returned a newly allocated list.
The leak was introduced in recent refactoring when llput was made to use validate_list_argument().
The problem is that the list NODE* that should have been gcref'd was also used to walk the list,
so NIL ended up being gcref'd instead.

Refactoring: inline untreeify()
After recent changes, it was only called from untreeify_body() and that function only called untreeify().
Therefore, the existence of both functions wasn't justified.
I retained the function with the stronger name.

Cleanup: Remove unused function untreeify_line.
This because unneeded when treeifying a proc's bodylist stopped changing each body line into a TREE.

Fix a comment

Update the baseline testlog based on a recent test pass.