1 files changed, 4 insertions, 1 deletions

diff --git a/src/fs/gnunet-fs-gtk_download-save-as.c b/src/fs/gnunet-fs-gtk_download-save-as.c
index 170f1bc9..1cd45e8d 100644
--- a/src/fs/gnunet-fs-gtk_download-save-as.c
+++ b/src/fs/gnunet-fs-gtk_download-save-as.c
@@ -84,7 +84,7 @@ save_as_dialog_free_download_context (struct DownloadContext *dc)
  * @param widget the dialog object
  * @param event the deletion event
  * @param user_data the 'structDownloadAsDialogContext' of the dialog
- * @return always FALSE
+ * @return always FALSE (destroy the window)
  */
 gboolean
 GNUNET_GTK_save_as_dialog_delete_event_cb (GtkWidget * widget, GdkEvent * event,
@@ -106,6 +106,7 @@ GNUNET_GTK_save_as_dialog_delete_event_cb (GtkWidget * widget, GdkEvent * event,
     GNUNET_free (dlc);
     return FALSE;
   }
+  /* FIXME-UGLY: this code should be in the 'save_as_dialog_response_cb' instead */
   GNUNET_free_non_null (dc->filename);
   dc->filename =
       GNUNET_GTK_filechooser_get_filename_utf8 (GTK_FILE_CHOOSER (dlc->dialog));
@@ -143,6 +144,8 @@ GNUNET_GTK_save_as_dialog_response_cb (GtkDialog * dialog,
   dlc->response = response_id;
   /* dialogs don't get delete-event the way normal windows do,
      call the handler manually */
+  /* FIXME-UGLY: this is a bit ugly; instead, move the code
+     to close the dialog nicely into here! */
   GNUNET_GTK_save_as_dialog_delete_event_cb (GTK_WIDGET (dialog), NULL,
                                              user_data);
   gtk_widget_destroy (GTK_WIDGET (dialog));