FMSLogo Code

Add a regression test that actually runs BYE (by running it in a child instance)

Include logodialogboxes.cpp in the GTK build.
This was primarily done to get support YESNOBOX to unblock the tests,
but they quickly blocked on GIFSAVE, which was immediately after the
YESNOBOX call.

Fix a bug that caused FORWARD to stop drawing lines after POLYVIEW was run
on GTK2 versions of wxWidgets (that is, on GNU/Linux). The bug was a
reference counting bug that caused POLYVIEW to destroy the turtle's pen.

Fix a bug that caused FORWARD to stop drawing lines after BITBLOCK
was run on GTK2 (GNU/Linux). In retrospect, the bug is obvious,
by maintaining a const reference to the wxPen that lbitblock() saves
and restores, the internal reference count is not incremented and
the wxPen is destroyed when it's replaced with a transparent pen,
leaving the saved reference to refer to an invalid pen.

Putting the saved pen into a new wxPen object fixes the problem.

I don't know why this problem doesn't affect MSW builds.

Remove an unneeded "#ifndef WX_PURE". The code it was compiling already had its body wrapped in a similar precompiler directive.

Remove a baffling call to wxDC::SetTextForeground() in logofill() which had no apparent purpose. I didn't observe any behavior change after removing it, nor did I find any documentation in the win32 ::ExtFloodFill() to suggest that the text color alters its behavior. This call has been there since the MSWLogo 6.5b, as ::SetTextColor().

Change the InvalidateRectangleOnScreen in lbitblock to use the overload that takes a wxRect, which was the reason it was created.

Move the logic for invalidating and updating the screen from lfill() to logofill() for parallelism with the other functions. Whenever a function draws to the memory device context, that function is also responsible for getting the change to the screen device context, either by making an identical change to the screen device context or by invalidating the screen.

This change also removes the initial screen update, which isn't needed the way logofill is implemented.

Port FILL and BITBLOCK from using win32 API to using wxWidgets API. This makes them portable across platforms and, in particular, makes them work on GNU/Linux.

Document the behavior of FILL when no fillmode input is given.