diff options
author | Tim Mayberry <mojofunk@gmail.com> | 2016-09-17 20:28:34 +1000 |
---|---|---|
committer | Tim Mayberry <mojofunk@gmail.com> | 2016-09-17 22:03:16 +1000 |
commit | 078889efa89ba2cb4be01108b80861acdc5113cb (patch) | |
tree | 2e0368f18a582f2099e1847922a4c923a5d07ae4 /gtk2_ardour/export_file_notebook.h | |
parent | ce16b5424fd05a72d1fe086caa3c1dd0a697916b (diff) |
Save changes when toggling checkboxes in ExportDialog
I used the Widget::on_hide method in the ExportFileDialog to defer the saving
of changes in state of the analysis and soundcloud-upload checkboxes as it was
not possible to save the format xml state directly from the
ToggleButton::toggled() signal as it created a recursive loop and also to
prevent saving the state more than once.
Even though the ExportProfileManager::FormatListChanged signal is no longer
emitted when saving format state and the crash no longer occurs without this
change. I think it is worth saving explicitily from in the toggle callbacks to
reduce the complexity of understanding what is taking place and when even if it
is less efficient.
There is definitely more opportunity for refactoring and redesign.
Diffstat (limited to 'gtk2_ardour/export_file_notebook.h')
-rw-r--r-- | gtk2_ardour/export_file_notebook.h | 4 |
1 files changed, 0 insertions, 4 deletions
diff --git a/gtk2_ardour/export_file_notebook.h b/gtk2_ardour/export_file_notebook.h index 3588197469..84d05c1025 100644 --- a/gtk2_ardour/export_file_notebook.h +++ b/gtk2_ardour/export_file_notebook.h @@ -100,8 +100,6 @@ class ExportFileNotebook : public Gtk::Notebook, public ARDOUR::SessionHandlePtr void analysis_changed (); void soundcloud_upload_changed (); - void on_hide (); - ARDOUR::ExportProfileManager::FormatStatePtr format_state; ARDOUR::ExportProfileManager::FilenameStatePtr filename_state; ManagerPtr profile_manager; @@ -124,8 +122,6 @@ class ExportFileNotebook : public Gtk::Notebook, public ARDOUR::SessionHandlePtr Gtk::Alignment tab_close_alignment; Gtk::Button tab_close_button; uint32_t tab_number; - - bool save_format_on_hide; }; }; |