Changed: QDir.cd already checks if directory exists, see #279

This commit is contained in:
kervala 2016-09-03 10:14:06 +02:00
parent e6bf79c85f
commit 4fa6f0dc22
2 changed files with 4 additions and 4 deletions

View file

@ -679,7 +679,7 @@ bool CConfigFile::foundTemporaryFiles(const QString &directory) const
// directory doesn't exist // directory doesn't exist
if (!dir.exists()) return false; if (!dir.exists()) return false;
if (!dir.cd("data") && dir.exists()) return false; if (!dir.cd("data")) return false;
QStringList filter; QStringList filter;
filter << "*.string_cache"; filter << "*.string_cache";
@ -695,7 +695,7 @@ bool CConfigFile::foundTemporaryFiles(const QString &directory) const
if (!dir.entryList(filter, QDir::Files).isEmpty()) return true; if (!dir.entryList(filter, QDir::Files).isEmpty()) return true;
// fonts directory is not needed anymore // fonts directory is not needed anymore
if (dir.exists("fonts.bnp") && dir.cd("fonts") && dir.exists()) return true; if (dir.exists("fonts.bnp") && !dir.cd("fonts")) return true;
return false; return false;
} }

View file

@ -46,7 +46,7 @@ bool CFilesCleaner::exec()
// directory doesn't exist // directory doesn't exist
if (!dir.exists()) return false; if (!dir.exists()) return false;
if (!dir.cd("data") && dir.exists()) return false; if (!dir.cd("data")) return false;
QStringList filter; QStringList filter;
filter << "*.string_cache"; filter << "*.string_cache";
@ -79,7 +79,7 @@ bool CFilesCleaner::exec()
} }
// fonts directory is not needed anymore if fonts.bnp exists // fonts directory is not needed anymore if fonts.bnp exists
if (dir.exists("fonts.bnp") && dir.cd("fonts") && dir.exists()) if (dir.exists("fonts.bnp") && dir.cd("fonts"))
{ {
dir.removeRecursively(); dir.removeRecursively();
} }