[Gnucash-changes] make sure that function doesn't return while structure is holding

[Linas Vepstas]

Log Message:
-----------
make sure that function doesn't return while structure is 
holding pointer to a stack variable.  Also, simplify logic.

Modified Files:
--------------
    gnucash/src/gnome-utils:
        dialog-transfer.c

Revision Data
-------------
Index: dialog-transfer.c
===================================================================
RCS file: /home/cvs/cvsroot/gnucash/src/gnome-utils/dialog-transfer.c,v
retrieving revision 1.21
retrieving revision 1.22
diff -Lsrc/gnome-utils/dialog-transfer.c -Lsrc/gnome-utils/dialog-transfer.c -u -r1.21 -r1.22
--- src/gnome-utils/dialog-transfer.c
+++ src/gnome-utils/dialog-transfer.c
@@ -2073,32 +2073,31 @@
 {
   gboolean result_ok = FALSE;
 
-  if( xferData )
-  {
-    xferData->result_p = &result_ok;
-    while( TRUE )
-    {
-      gnome_dialog_run( GNOME_DIALOG(xferData->dialog) );
-
-        /* See if the dialog is still there.  For various reasons, the
-         * user could have hit OK but remained in the dialog.  We don't
-         * want to return processing back to anyone else until we clear
-         * off this dialog, so if the dialog is still there we'll just
-         * run it again.
-         */
+  if( NULL == xferData ) return FALSE;
   
-      DEBUG("find component");
-      if( !gnc_find_first_gui_component( DIALOG_TRANSFER_CM_CLASS,
-                                           find_xfer, xferData ) )
-        {
-          /* no more dialog, and OK was clicked, so assume it's all good */
-          return( result_ok );
-        }
-        /* else run the dialog again */
-    }
+  xferData->result_p = &result_ok;
+  while( TRUE )
+  {
+    gnome_dialog_run( GNOME_DIALOG(xferData->dialog) );
+
+    /* See if the dialog is still there.  For various reasons, the
+     * user could have hit OK but remained in the dialog.  We don't
+     * want to return processing back to anyone else until we clear
+     * off this dialog, so if the dialog is still there we'll just
+     * run it again.
+     */
+
+    DEBUG("find component");
+    if( !gnc_find_first_gui_component( DIALOG_TRANSFER_CM_CLASS,
+                                         find_xfer, xferData ) )
+      {
+        /* no more dialog, and OK was clicked, so assume it's all good */
+        xferData->result_p = NULL;
+        return( result_ok );
+      }
+      /* else run the dialog again */
   }
-    
-  return( FALSE );
+  return FALSE;
 }