aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorJ-P Nurmi <jpnurmi@qt.io>2016-04-23 12:50:37 +0200
committerJ-P Nurmi <jpnurmi@qt.io>2016-04-25 03:48:00 +0000
commitdd0a29bb332b8ed0acfdf72ab2072be3bbd84006 (patch)
treea827554170370d1dbba2d11455d4f6fb2ecb0d8e
parent6d3a0e7e65b874af5ebb7edf117a2b5d1f436aa3 (diff)
QQuickStyleSelector: re-use QFileSelectorPrivate::platformSelectors()
Better not to duplicate this code, because it will easily get out of sync. Change-Id: I9a5eb97cee2c3d15ebc4af157cee3d54d4a9e700 Reviewed-by: Mitch Curtis <mitch.curtis@theqtcompany.com>
-rw-r--r--src/quickcontrols2/qquickstyleselector.cpp37
1 files changed, 3 insertions, 34 deletions
diff --git a/src/quickcontrols2/qquickstyleselector.cpp b/src/quickcontrols2/qquickstyleselector.cpp
index 6484da06..cd89ba95 100644
--- a/src/quickcontrols2/qquickstyleselector.cpp
+++ b/src/quickcontrols2/qquickstyleselector.cpp
@@ -42,6 +42,7 @@
#include <QtCore/qsysinfo.h>
#include <QtCore/qlocale.h>
+#include <QtCore/private/qfileselector_p.h>
#include <QtGui/private/qguiapplication_p.h>
QT_BEGIN_NAMESPACE
@@ -55,42 +56,10 @@ static bool isLocalScheme(const QString &scheme)
return local;
}
-// similar, but not identical to QSysInfo::osType
-static QStringList platformSelectors()
-{
- static QStringList selectors;
- if (!selectors.isEmpty())
- return selectors;
-
-#if defined(Q_OS_WIN)
- // can't fall back to QSysInfo because we need both "winphone" and "winrt" for the Windows Phone case
- selectors << QStringLiteral("windows");
- selectors << QSysInfo::kernelType(); // "wince" and "winnt"
-# if defined(Q_OS_WINRT)
- selectors << QStringLiteral("winrt");
-# if defined(Q_OS_WINPHONE)
- selectors << QStringLiteral("winphone");
-# endif
-# endif
-#elif defined(Q_OS_UNIX)
- selectors << QStringLiteral("unix");
-# if !defined(Q_OS_ANDROID) && !defined(Q_OS_BLACKBERRY)
- // we don't want "linux" for Android or "qnx" for Blackberry here
- selectors << QSysInfo::kernelType();
-# ifdef Q_OS_MAC
- selectors << QStringLiteral("mac"); // compatibility, since kernelType() is "darwin"
-# endif
-# endif
- QString productName = QSysInfo::productType();
- if (productName != QLatin1String("unknown"))
- selectors << productName; // "opensuse", "fedora", "osx", "ios", "blackberry", "android"
-#endif
- return selectors;
-}
-
static QStringList allSelectors(bool includeStyle)
{
- QStringList selectors = platformSelectors();
+ static const QStringList platformSelectors = QFileSelectorPrivate::platformSelectors();
+ QStringList selectors = platformSelectors;
selectors += QLocale().name();
if (includeStyle) {
QString style = QQuickStyle::name();