Attempt to fix the Gtk-CRITICAL **: file gtkwidget.c: line 3892:

assertion `GTK_IS_WIDGET (widget)' failed when quitting.

Cause. The Menubar item callback was not the same as the gtk_signal_connect() callback.

src/zmapControl/zmapControlWindow.c

@@ -26,9 +26,9 @@
  *              
  * Exported functions: See zmapTopWindow_P.h
  * HISTORY:
- * Last edited: Feb 21 15:40 2006 (edgrif)
+ * Last edited: Feb 24 12:14 2006 (rds)
  * Created: Fri May  7 14:43:28 2004 (edgrif)
- * CVS info:   $Id: zmapControlWindow.c,v 1.20 2006-02-23 16:13:35 edgrif Exp $
+ * CVS info:   $Id: zmapControlWindow.c,v 1.21 2006-02-24 12:16:09 rds Exp $
  *-------------------------------------------------------------------
  */
 
@@ -206,17 +206,30 @@ static GtkWidget *makeStatusPanel(ZMap zmap)
 }
 
 
-
+/* Probably needs renaming to exitCB, Need to check with Ed as not
+ * 100% on all possiblities, esp threads... */
 static void quitCB(GtkWidget *widget, gpointer cb_data)
 {
   ZMap zmap = (ZMap)cb_data ;
 
+#warning THIS NEEDS CHECKING (RDS)
+#ifdef CAUSE_OF_GTK_CRITICAL_ISSUE
   /* this is hacky, I don't like this, if we don't do this then end up trying to kill a
    * non-existent top level window because gtk already seems to have done this...  */
-  zmap->toplevel = NULL ;
-
+  zmap->toplevel = NULL ; 
+  
   zmapControlTopLevelKillCB(zmap) ;
-
+#endif /* CAUSE_OF_GTK_CRITICAL_ISSUE */
+
+  printf("Ed....\nI'm quitCB() in %s:%d\n"
+         "I think this is where the 'Gtk-CRITICAL **:"
+         " gtk_widget_set_sensitive: assertion"
+         " `GTK_IS_WIDGET (widget)' failed' issue is,"
+         " can you check this.\n", 
+         __FILE__, __LINE__);
+  /* So now we call this which appears to cure the issue. Assuming
+   * threads all get cleaned up in this */
+  (*(zmap->zmap_cbs_G->exit))(zmap, zmap->app_data) ;
 
   return ;
 }