diff options
author | Arnaud Lacombe <lacombar@gmail.com> | 2011-05-24 14:16:18 -0400 |
---|---|---|
committer | Michal Marek <mmarek@suse.cz> | 2011-05-25 09:05:07 -0400 |
commit | d49e46875c11a09e80e76c66db90710369b8fe12 (patch) | |
tree | 7a5b6400e49cffbcb708ac562cf50735a2c0d1d2 /scripts/kconfig | |
parent | bdebd4892e05cc9068659f25af33c6b322034eb2 (diff) |
xconfig: merge code path to conf_write()
Avoid to have multiple path saving the config. This fixes an error check
miss when the window is being closed and the user requested the config
to be written.
Reported-by: Hiromu Yakura <hiromu1996@gmail.com>
Pointed-out-by: Michal Marek <mmarek@suse.cz>
Signed-off-by: Michal Marek <mmarek@suse.cz>
Diffstat (limited to 'scripts/kconfig')
-rw-r--r-- | scripts/kconfig/qconf.cc | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/scripts/kconfig/qconf.cc b/scripts/kconfig/qconf.cc index 06dd2e33581d..c2796b866f8f 100644 --- a/scripts/kconfig/qconf.cc +++ b/scripts/kconfig/qconf.cc | |||
@@ -1489,8 +1489,7 @@ void ConfigMainWindow::saveConfigAs(void) | |||
1489 | QString s = Q3FileDialog::getSaveFileName(conf_get_configname(), NULL, this); | 1489 | QString s = Q3FileDialog::getSaveFileName(conf_get_configname(), NULL, this); |
1490 | if (s.isNull()) | 1490 | if (s.isNull()) |
1491 | return; | 1491 | return; |
1492 | if (conf_write(QFile::encodeName(s))) | 1492 | saveConfig(); |
1493 | QMessageBox::information(this, "qconf", _("Unable to save configuration!")); | ||
1494 | } | 1493 | } |
1495 | 1494 | ||
1496 | void ConfigMainWindow::searchConfig(void) | 1495 | void ConfigMainWindow::searchConfig(void) |
@@ -1643,7 +1642,7 @@ void ConfigMainWindow::closeEvent(QCloseEvent* e) | |||
1643 | mb.setButtonText(QMessageBox::Cancel, _("Cancel Exit")); | 1642 | mb.setButtonText(QMessageBox::Cancel, _("Cancel Exit")); |
1644 | switch (mb.exec()) { | 1643 | switch (mb.exec()) { |
1645 | case QMessageBox::Yes: | 1644 | case QMessageBox::Yes: |
1646 | conf_write(NULL); | 1645 | saveConfig(); |
1647 | case QMessageBox::No: | 1646 | case QMessageBox::No: |
1648 | e->accept(); | 1647 | e->accept(); |
1649 | break; | 1648 | break; |