QStandardPaths: Use forward slash consistently.
Task-number: QTBUG-29249 Change-Id: I027f9ae18544dc47e1378214244487c8a5ae704c Reviewed-by: Frederik Gladhorn <frederik.gladhorn@digia.com> Reviewed-by: Thiago Macieira <thiago.macieira@intel.com>
This commit is contained in:
parent
f136701bc5
commit
cd1e245877
@ -160,10 +160,10 @@ QString QStandardPaths::writableLocation(StandardLocation type)
|
||||
// Although Microsoft has a Cache key it is a pointer to IE's cache, not a cache
|
||||
// location for everyone. Most applications seem to be using a
|
||||
// cache directory located in their AppData directory
|
||||
return writableLocation(DataLocation) + QLatin1String("\\cache");
|
||||
return writableLocation(DataLocation) + QLatin1String("/cache");
|
||||
|
||||
case GenericCacheLocation:
|
||||
return writableLocation(GenericDataLocation) + QLatin1String("\\cache");
|
||||
return writableLocation(GenericDataLocation) + QLatin1String("/cache");
|
||||
|
||||
case RuntimeLocation:
|
||||
case HomeLocation:
|
||||
|
@ -45,6 +45,7 @@
|
||||
#include <qstandardpaths.h>
|
||||
#include <qfileinfo.h>
|
||||
#include <qsysinfo.h>
|
||||
#include <qregexp.h>
|
||||
|
||||
#ifdef Q_OS_UNIX
|
||||
#include <unistd.h>
|
||||
@ -72,6 +73,7 @@ private slots:
|
||||
void testCustomRuntimeDirectory();
|
||||
void testAllWritableLocations_data();
|
||||
void testAllWritableLocations();
|
||||
void testCleanPath();
|
||||
|
||||
private:
|
||||
#ifdef Q_XDG_PLATFORM
|
||||
@ -430,6 +432,18 @@ void tst_qstandardpaths::testAllWritableLocations()
|
||||
QCOMPARE(loc.endsWith(QLatin1Char('/')), false);
|
||||
}
|
||||
|
||||
void tst_qstandardpaths::testCleanPath()
|
||||
{
|
||||
const QRegExp filter(QStringLiteral("\\\\"));
|
||||
QVERIFY(filter.isValid());
|
||||
for (int i = 0; i <= QStandardPaths::GenericCacheLocation; ++i) {
|
||||
const QStringList paths = QStandardPaths::standardLocations(QStandardPaths::StandardLocation(i));
|
||||
QVERIFY2(paths.filter(filter).isEmpty(),
|
||||
qPrintable(QString::fromLatin1("Backslash found in %1 %2")
|
||||
.arg(i).arg(paths.join(QLatin1Char(',')))));
|
||||
}
|
||||
}
|
||||
|
||||
QTEST_MAIN(tst_qstandardpaths)
|
||||
|
||||
#include "tst_qstandardpaths.moc"
|
||||
|
Loading…
Reference in New Issue
Block a user