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

--HG--
branch : develop

code/ryzom/tools/client/ryzom_installer/src/configfile.cpp

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

code/ryzom/tools/client/ryzom_installer/src/filescleaner.cpp

@@ -46,7 +46,7 @@ bool CFilesCleaner::exec()
 	// directory doesn't exist
 	if (!dir.exists()) return false;
 
-	if (!dir.cd("data") && dir.exists()) return false;
+	if (!dir.cd("data")) return false;
 
 	QStringList filter;
 	filter << "*.string_cache";
@@ -79,7 +79,7 @@ bool CFilesCleaner::exec()
 	}
 
 	// 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();
 	}