From 78c5aa9f071ab56e56bf760e3e7fc7f0f0d1b1ff Mon Sep 17 00:00:00 2001 From: Dirk Hohndel Date: Mon, 10 Sep 2012 12:27:00 -0700 Subject: Change behavior for the existing filename Previously we always picked the last file that was openend as the file name to save to. That seems counterintuitive when importing files or when opening multiple files. Especially if Subsurface was executed without a file on the command line and we are using the default file. Now we only remember a file name if it was the first one to ever be openend or if it was used in save-as. Signed-off-by: Dirk Hohndel --- dive.h | 2 +- gtk-gui.c | 10 +++++----- main.c | 2 +- parse-xml.c | 5 ++--- 4 files changed, 9 insertions(+), 10 deletions(-) diff --git a/dive.h b/dive.h index 746b94093..af141bbde 100644 --- a/dive.h +++ b/dive.h @@ -379,7 +379,7 @@ static inline struct dive *get_dive(unsigned int nr) extern void parse_xml_init(void); extern void parse_xml_buffer(const char *url, const char *buf, int size, GError **error); -extern void set_filename(const char *filename); +extern void set_filename(const char *filename, gboolean force); extern void parse_file(const char *filename, GError **error); diff --git a/gtk-gui.c b/gtk-gui.c index 1d5fc4cc9..6666f5560 100644 --- a/gtk-gui.c +++ b/gtk-gui.c @@ -233,7 +233,7 @@ static void file_save_as(GtkWidget *w, gpointer data) if (filename){ save_dives(filename); - set_filename(filename); + set_filename(filename, TRUE); g_free(filename); mark_divelist_changed(FALSE); } @@ -1419,11 +1419,11 @@ void update_progressbar_text(progressbar_t *progress, const char *text) gtk_progress_bar_set_text(GTK_PROGRESS_BAR(progress->bar), text); } -void set_filename(const char *filename) +void set_filename(const char *filename, gboolean force) { - if (existing_filename) - free(existing_filename); - existing_filename = NULL; + if (!force && existing_filename) + return; + free(existing_filename); if (filename) existing_filename = strdup(filename); } diff --git a/main.c b/main.c index a2c24ec99..01306b1bf 100644 --- a/main.c +++ b/main.c @@ -243,7 +243,7 @@ int main(int argc, char **argv) parse_file(filename, &error); /* don't report errors - this file may not exist, but make sure we remember this as the filename in use */ - set_filename(filename); + set_filename(filename, FALSE); } report_dives(imported); diff --git a/parse-xml.c b/parse-xml.c index 1c4db7d9d..a4ae81b25 100644 --- a/parse-xml.c +++ b/parse-xml.c @@ -1475,9 +1475,8 @@ void parse_xml_buffer(const char *url, const char *buffer, int size, GError **er } return; } - /* we assume that the last (or only) filename passed as argument is a - * great filename to use as default when saving the dives */ - set_filename(url); + /* remember, if necessary, that this is the filename to store to */ + set_filename(url, FALSE); reset_all(); dive_start(); #ifdef XSLT -- cgit v1.2.3-70-g09d2