[gnue] r9000 - trunk/gnue-forms/src/uidrivers/wx26/widgets

[reinhard]

Author: reinhard
Date: 2006-11-08 07:07:14 -0600 (Wed, 08 Nov 2006)
New Revision: 9000

Modified:
   trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py
Log:
Add reference to issue and some extra comments to workarounds.


Modified: trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py
===================================================================
--- trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py 2006-11-08 12:54:41 UTC 
(rev 8999)
+++ trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py 2006-11-08 13:07:14 UTC 
(rev 9000)
@@ -289,12 +289,19 @@
             # this event, meaning the form is being destroyed. For some reason,
             # this doesn't work, so we destroy the form manually.
             # event.Skip()
-            #
-            # We have to call Show() here to make sure the main loop can exit
-            # cleanly.  This workaround is obsolete with wx2.7+
+
+            # FIXME: Workaround for issue132: Destroying Windows that are not
+            # visible at the time of being destroyed does not work in wx2.6.x.
+            # This workaround shoud be obsolete with wx2.7+
             event.GetEventObject().Show()
+
             event.GetEventObject().Destroy()
-            # After destroying the window we do not have a main_window anymore
+
+            # FIXME: Workaround for issue131: Calling wx.EndBusyCursor()
+            # at a time when all forms have been destroyed generates a segfault
+            # if the last form that has been shown is open on a non-first page.
+            # So we set main_window to None when destroying the main window and
+            # don't call wx.EndBusyCursor() if main_window is None.
             self.main_window = None
 
     # -------------------------------------------------------------------------
@@ -332,9 +339,11 @@
         """
         Display the normal mouse cursor on all windows of the application.
         """
-        # If a trigger closes the form, this is called after the form is
-        # closed, which would result in a segfault. So we have to check whether
-        # the form is still available.
+        # FIXME: Workaround for issue131: Calling wx.EndBusyCursor()
+        # at a time when all forms have been destroyed generates a segfault
+        # if the last form that has been shown is open on a non-first page.
+        # So we set main_window to None when destroying the main window and
+        # don't call wx.EndBusyCursor() if main_window is None.
         if self.main_window and wx.IsBusy():
             wx.EndBusyCursor()