[gnue] r8952 - in trunk/gnue-forms/src/uidrivers/wx26: . widgets

[reinhard]

Author: reinhard
Date: 2006-10-30 08:51:47 -0600 (Mon, 30 Oct 2006)
New Revision: 8952

Modified:
   trunk/gnue-forms/src/uidrivers/wx26/UIdriver.py
   trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py
Log:
Updated comments.


Modified: trunk/gnue-forms/src/uidrivers/wx26/UIdriver.py
===================================================================
--- trunk/gnue-forms/src/uidrivers/wx26/UIdriver.py     2006-10-30 14:23:48 UTC 
(rev 8951)
+++ trunk/gnue-forms/src/uidrivers/wx26/UIdriver.py     2006-10-30 14:51:47 UTC 
(rev 8952)
@@ -228,8 +228,6 @@
     # loop automatically. However, this does not work in cases where a dialog
     # box is shown while closing (asking whether to save changes or not), so we
     # end the main loop manually.
-    # FIXME (issue131): This happens to also fix the segfault when closing a
-    # form while not on the first tab.
     wx.GetApp().ExitMainLoop()
 
     assert gLeave(6)

Modified: trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py
===================================================================
--- trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py 2006-10-30 14:23:48 UTC 
(rev 8951)
+++ trunk/gnue-forms/src/uidrivers/wx26/widgets/form.py 2006-10-30 14:51:47 UTC 
(rev 8952)
@@ -323,6 +323,9 @@
         """
         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 visible.
         if wx.IsBusy() and self.main_window.IsShown():
             wx.EndBusyCursor()