FMSLogo Code

Refactor the "accept ownership of erract" tests to use SIMPLEREPORTTESTNOCATCH.
This removes some duplicated code.

Refactoring: Simplify lbitsave() by using get_int() instead of integer_arg() and getint().

Refactoring: Simplify lgifsave() by advancing args to the next node as arguments are read instead of having long chains of cdr.
This makes the code shorter and more readable.

Refactoring: Simplify lgifsave() by using get_int() instead of integer_arg() and getint().

Update the expected test log with the output from test cases that were recently added.

Rework int_arg() so that it doesn't modify the arguments list.
This is part of a larger effort to simplify the code base by reducing the number of side effects.

Rename CLogoCodeCtrl::Replace to CLogoCodeCtrl::ReplaceText and make it non-virtual to fix a -Woverloaded-virtual warning.

Add missing "const" qualifier to CanCut and CanCopy so that they override the virtual functions in wxTextCtrl.
This required a little rework to a helper function IsTextSelected() to make it const, as well.
This fixes two -Woverloaded-virtual warnings in gcc.

Refactoring: Change "int macro_flag" to "bool is_macro" in define_helper.
The caller that passed the third state of -1 now calls make_procnode_from_body_list().
This makes the code simpler.

Refactoring: Extract make_procnode_from_body_list() from define_helper.
It is the only portion of define_helper() that anonymous_function uses.
This is a step toward removing the tristate "int macro_flag" from define_helper.