1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QStyleSheetStyle: avoid double lookups to QRenderRule::styleHints

Browse Source 
Avoid double lookups to QRenderRule::styleHints by simply checking if
the returned QVariant from styleHint() is default-constructed.

Change-Id: I4ca657662c2af66582fb4fc7c2ed1b77258ab050
Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
(cherry picked from commit f1fd15b4549a740dbd0abff44a43b551eef12d31)
This commit is contained in:
Christian Ehrlicher 2024-11-15 14:05:19 +01:00
parent bede1100fb
commit 6855403445
1 changed files with 15 additions and 13 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

28
src/widgets/styles/qstylesheetstyle.cpp
View File

@ -592,7 +592,6 @@ public:
return csz;
}
bool hasStyleHint(QLatin1StringView sh) const { return styleHints.contains(sh); }
QVariant styleHint(QLatin1StringView sh) const { return styleHints.value(sh); }
void fixupBorder(int);
@ -5236,11 +5235,12 @@ int QStyleSheetStyle::pixelMetric(PixelMetric m, const QStyleOption *opt, const
case PM_TabBarIconSize:
case PM_MessageBoxIconSize:
case PM_ButtonIconSize:
case PM_SmallIconSize:
if (rule.hasStyleHint("icon-size"_L1))
return rule.styleHint("icon-size"_L1).toSize().width();
case PM_SmallIconSize: {
const auto styleHint = rule.styleHint("icon-size"_L1);
if (styleHint.isValid() && styleHint.canConvert<QSize>())
return styleHint.toSize().width();
break;
}
case PM_DockWidgetTitleMargin: {
QRenderRule subRule = renderRule(w, opt, PseudoElement_DockWidgetTitle);
if (!subRule.hasBox())
@ -5643,9 +5643,9 @@ QIcon QStyleSheetStyle::standardIcon(StandardPixmap standardIcon, const QStyleOp
RECURSION_GUARD(return baseStyle()->standardIcon(standardIcon, opt, w))
const auto s = propertyNameForStandardPixmap(standardIcon);
if (!s.isEmpty()) {
QRenderRule rule = renderRule(w, opt);
if (rule.hasStyleHint(s))
return qvariant_cast<QIcon>(rule.styleHint(s));
const auto styleHint = renderRule(w, opt).styleHint(s);
if (styleHint.isValid() && styleHint.canConvert<QIcon>())
return qvariant_cast<QIcon>(styleHint);
}
return baseStyle()->standardIcon(standardIcon, opt, w);
}
@ -5661,9 +5661,9 @@ QPixmap QStyleSheetStyle::standardPixmap(StandardPixmap standardPixmap, const QS
RECURSION_GUARD(return baseStyle()->standardPixmap(standardPixmap, opt, w))
const auto s = propertyNameForStandardPixmap(standardPixmap);
if (!s.isEmpty()) {
QRenderRule rule = renderRule(w, opt);
if (rule.hasStyleHint(s)) {
QIcon icon = qvariant_cast<QIcon>(rule.styleHint(s));
const auto styleHint = renderRule(w, opt).styleHint(s);
if (styleHint.isValid() && styleHint.canConvert<QIcon>()) {
QIcon icon = qvariant_cast<QIcon>(styleHint);
const auto dpr = w ? w->devicePixelRatio() : qApp->devicePixelRatio();
return icon.pixmap(QSize(16, 16), dpr);
}
@ -5782,8 +5782,10 @@ int QStyleSheetStyle::styleHint(StyleHint sh, const QStyleOption *opt, const QWi
break;
default: break;
}
if (!s.isEmpty() && rule.hasStyleHint(s)) {
return rule.styleHint(s).toInt();
if (!s.isEmpty()) {
const auto styleHint = rule.styleHint(s);
if (styleHint.isValid() && styleHint.canConvert<int>())
return styleHint.toInt();
}
return baseStyle()->styleHint(sh, opt, w, shret);