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 | |
| 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')
| -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; |
