diff options
author | Berthold Stoeger <bstoeger@mail.tuwien.ac.at> | 2017-12-04 08:49:28 +0100 |
---|---|---|
committer | Lubomir I. Ivanov <neolit123@gmail.com> | 2017-12-04 18:35:44 +0100 |
commit | 0a2e53caf27b30186f8492fcfb6efa61ca7808b1 (patch) | |
tree | d9b8db120031bc6c8ecc139ad143a1096642341b /desktop-widgets/divelogexportdialog.cpp | |
parent | ed92b60bfe249e2ec030c8e103e545dec854aba9 (diff) | |
download | subsurface-0a2e53caf27b30186f8492fcfb6efa61ca7808b1.tar.gz |
Use QDir::exists() instead of QDir::setCurrent() to check for existence
Don't change into a directory just to see if it exists.
Remove unnecessary braces of one of the changed if statements.
Signed-off-by: Berthold Stoeger <bstoeger@mail.tuwien.ac.at>
Diffstat (limited to 'desktop-widgets/divelogexportdialog.cpp')
-rw-r--r-- | desktop-widgets/divelogexportdialog.cpp | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/desktop-widgets/divelogexportdialog.cpp b/desktop-widgets/divelogexportdialog.cpp index ed908e937..96417c1bc 100644 --- a/desktop-widgets/divelogexportdialog.cpp +++ b/desktop-widgets/divelogexportdialog.cpp @@ -127,9 +127,8 @@ void DiveLogExportDialog::on_buttonBox_accepted() settings.beginGroup("FileDialog"); if (settings.contains("LastDir")) { - if (QDir::setCurrent(settings.value("LastDir").toString())) { + if (QDir(settings.value("LastDir").toString()).exists()) lastDir = settings.value("LastDir").toString(); - } } settings.endGroup(); |