diff options
author | J-P Nurmi <jpnurmi@qt.io> | 2016-04-23 12:50:37 +0200 |
---|---|---|
committer | J-P Nurmi <jpnurmi@qt.io> | 2016-04-25 03:48:00 +0000 |
commit | dd0a29bb332b8ed0acfdf72ab2072be3bbd84006 (patch) | |
tree | a827554170370d1dbba2d11455d4f6fb2ecb0d8e | |
parent | 6d3a0e7e65b874af5ebb7edf117a2b5d1f436aa3 (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.cpp | 37 |
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(); |