Members of the KDE Community are recommended to subscribe to the kde-community mailing list at https://mail.kde.org/mailman/listinfo/kde-community to allow them to participate in important discussions and receive other important announcements

Commit c96efc7c authored by Fabian Vogt's avatar Fabian Vogt

Fix build with Qt 5.10

Summary: QStaticText's constructor with const QString & as argument is now explicit.

Test Plan:
Did not build before, now does.

Note that this might change the behaviour as the other properties of m_text are now
kept, which was not the case before. I'm not sure what the expected behaviour here
is supposed to be.

Reviewers: #dolphin, #kde_applications, cfeck

Reviewed By: cfeck

Subscribers: cfeck, #kde_applications, #dolphin

Differential Revision: https://phabricator.kde.org/D8075
parent 478f404b
......@@ -101,7 +101,7 @@ void KStandardItemListGroupHeader::updateCache()
const qreal maxWidth = size().width() - 4 * styleOption().padding;
if (role() == "rating") {
m_text = QString(); // krazy:exlude=nullstrassign
m_text.setText(QString());
const qreal height = styleOption().fontMetrics.ascent();
const QSizeF pixmapSize(qMin(height * 5, maxWidth), height);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment