Only write settings to disk once in config dialog

src/citra_qt/configure_audio.cpp

@@ -53,7 +53,7 @@ void ConfigureAudio::setConfiguration() {
     ui->audio_device_combo_box->setCurrentIndex(new_device_index);
 }
 
-void ConfigureAudio::applyConfiguration() {
+void ConfigureAudio::applyConfiguration(bool commit_settings_file) {
     Settings::values.sink_id =
         ui->output_sink_combo_box->itemText(ui->output_sink_combo_box->currentIndex())
             .toStdString();
@@ -61,8 +61,10 @@ void ConfigureAudio::applyConfiguration() {
     Settings::values.audio_device_id =
         ui->audio_device_combo_box->itemText(ui->audio_device_combo_box->currentIndex())
             .toStdString();
+    if (commit_settings_file) {
         Settings::Apply();
     }
+}
 
 void ConfigureAudio::updateAudioDevices(int sink_index) {
     ui->audio_device_combo_box->clear();

src/citra_qt/configure_audio.h

@@ -18,7 +18,7 @@ public:
     explicit ConfigureAudio(QWidget* parent = nullptr);
     ~ConfigureAudio();
 
-    void applyConfiguration();
+    void applyConfiguration(bool commit_settings_file = true);
 
 public slots:
     void updateAudioDevices(int sink_index);

src/citra_qt/configure_debug.cpp

@@ -19,8 +19,10 @@ void ConfigureDebug::setConfiguration() {
     ui->gdbport_spinbox->setValue(Settings::values.gdbstub_port);
 }
 
-void ConfigureDebug::applyConfiguration() {
+void ConfigureDebug::applyConfiguration(bool commit_settings_file) {
     Settings::values.use_gdbstub = ui->toggle_gdbstub->isChecked();
     Settings::values.gdbstub_port = ui->gdbport_spinbox->value();
+    if (commit_settings_file) {
         Settings::Apply();
     }
+}

src/citra_qt/configure_debug.h

@@ -18,7 +18,7 @@ public:
     explicit ConfigureDebug(QWidget* parent = nullptr);
     ~ConfigureDebug();
 
-    void applyConfiguration();
+    void applyConfiguration(bool commit_settings_file = true);
 
 private:
     void setConfiguration();

src/citra_qt/configure_dialog.cpp

@@ -17,11 +17,15 @@ ConfigureDialog::~ConfigureDialog() {}
 void ConfigureDialog::setConfiguration() {}
 
 void ConfigureDialog::applyConfiguration() {
-    ui->generalTab->applyConfiguration();
+    // apply each, but don't commit the ini file (and trigger
-    ui->systemTab->applyConfiguration();
+    //  associated changes) until the end
-    ui->inputTab->applyConfiguration();
+    ui->generalTab->applyConfiguration(false);
-    ui->graphicsTab->applyConfiguration();
+    ui->inputTab->applyConfiguration(false);
-    ui->audioTab->applyConfiguration();
+    ui->graphicsTab->applyConfiguration(false);
-    ui->debugTab->applyConfiguration();
+    ui->audioTab->applyConfiguration(false);
+    ui->debugTab->applyConfiguration(false);
+
     Settings::Apply();
+    // system settings doesn't actually put settings in the ini file
+    ui->systemTab->applyConfiguration();
 }

src/citra_qt/configure_general.cpp

@@ -28,10 +28,12 @@ void ConfigureGeneral::setConfiguration() {
     ui->region_combobox->setCurrentIndex(Settings::values.region_value + 1);
 }
 
-void ConfigureGeneral::applyConfiguration() {
+void ConfigureGeneral::applyConfiguration(bool commit_settings_file) {
     UISettings::values.gamedir_deepscan = ui->toggle_deepscan->isChecked();
     UISettings::values.confirm_before_closing = ui->toggle_check_exit->isChecked();
     Settings::values.region_value = ui->region_combobox->currentIndex() - 1;
     Settings::values.use_cpu_jit = ui->toggle_cpu_jit->isChecked();
+    if (commit_settings_file) {
         Settings::Apply();
     }
+}

src/citra_qt/configure_general.h

@@ -18,7 +18,7 @@ public:
     explicit ConfigureGeneral(QWidget* parent = nullptr);
     ~ConfigureGeneral();
 
-    void applyConfiguration();
+    void applyConfiguration(bool commit_settings_file = true);
 
 private:
     void setConfiguration();

src/citra_qt/configure_graphics.cpp

@@ -101,7 +101,7 @@ void ConfigureGraphics::setConfiguration() {
     ui->swap_screen->setChecked(Settings::values.swap_screen);
 }
 
-void ConfigureGraphics::applyConfiguration() {
+void ConfigureGraphics::applyConfiguration(bool commit_settings_file) {
     Settings::values.use_hw_renderer = ui->toggle_hw_renderer->isChecked();
     Settings::values.use_shader_jit = ui->toggle_shader_jit->isChecked();
     Settings::values.resolution_factor =
@@ -111,5 +111,7 @@ void ConfigureGraphics::applyConfiguration() {
     Settings::values.layout_option =
         static_cast<Settings::LayoutOption>(ui->layout_combobox->currentIndex());
     Settings::values.swap_screen = ui->swap_screen->isChecked();
+    if (commit_settings_file) {
         Settings::Apply();
     }
+}

src/citra_qt/configure_graphics.h

@@ -18,7 +18,7 @@ public:
     explicit ConfigureGraphics(QWidget* parent = nullptr);
     ~ConfigureGraphics();
 
-    void applyConfiguration();
+    void applyConfiguration(bool commit = true);
 
 private:
     void setConfiguration();

src/citra_qt/configure_input.cpp

@@ -112,14 +112,16 @@ ConfigureInput::ConfigureInput(QWidget* parent)
     ui->buttonCStickRight->setEnabled(false);
 }
 
-void ConfigureInput::applyConfiguration() {
+void ConfigureInput::applyConfiguration(bool commit_settings_file) {
     std::transform(buttons_param.begin(), buttons_param.end(), Settings::values.buttons.begin(),
                    [](const Common::ParamPackage& param) { return param.Serialize(); });
     std::transform(analogs_param.begin(), analogs_param.end(), Settings::values.analogs.begin(),
                    [](const Common::ParamPackage& param) { return param.Serialize(); });
 
+    if (commit_settings_file) {
         Settings::Apply();
     }
+}
 
 void ConfigureInput::loadConfiguration() {
     std::transform(Settings::values.buttons.begin(), Settings::values.buttons.end(),

src/citra_qt/configure_input.h

@@ -30,7 +30,7 @@ public:
     explicit ConfigureInput(QWidget* parent = nullptr);
 
     /// Save all button configurations to settings file
-    void applyConfiguration();
+    void applyConfiguration(bool commit_settings_file = true);
 
 private:
     std::unique_ptr<Ui::ConfigureInput> ui;