diff options
| author | Tim Mayberry <mojofunk@gmail.com> | 2016-09-17 19:20:06 +1000 |
|---|---|---|
| committer | Tim Mayberry <mojofunk@gmail.com> | 2016-09-17 22:03:16 +1000 |
| commit | ce16b5424fd05a72d1fe086caa3c1dd0a697916b (patch) | |
| tree | 1ad2e000bf946ab429b0a9a6f639000ad92aa045 /libs/ardour/export_profile_manager.cc | |
| parent | ac4ad28b4f3c89f6b380bdc5ea7affe2d3e05247 (diff) | |
Fix crash when ExportDialog is hidden
The issue is that when ExportDialog is destroyed
ExportFileNotebook::FilePage::on_hide is called which in turn calls
ExportProfileManager::save_format_to_disk() in order to save changes to the
analysis and soundcloud upload checkboxes. This then causes the
ExportProfileManager::FormatListChanged signal to be emitted, which in all
other cases is emitted when the format list is modified so that the GUI can
rebuild the format selector menu in ExportFormatSelector::update_format_list
when a format is added or removed.
The problem when doing this in the destructor is that some of the widgets have
already been destroyed, specifically the issue was in
ExportFileNotebook::update_soundcloud_upload trying to access the
soundcloud_selector member that had already had its destructor called.
As it is not necessary to call this signal in the first place and it just
causes unnecessary GUI updates remove the signal emission.
Diffstat (limited to 'libs/ardour/export_profile_manager.cc')
| -rw-r--r-- | libs/ardour/export_profile_manager.cc | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/libs/ardour/export_profile_manager.cc b/libs/ardour/export_profile_manager.cc index 4eb2e62b0b..94099faa26 100644 --- a/libs/ardour/export_profile_manager.cc +++ b/libs/ardour/export_profile_manager.cc @@ -633,7 +633,6 @@ ExportProfileManager::save_format_to_disk (ExportFormatSpecPtr format) tree.write(); } - FormatListChanged (); return new_path; } |