]> cloud.milkyroute.net Git - dolphin.git/commitdiff
Fix build with Qt 5.10
authorFabian Vogt <fabian@ritter-vogt.de>
Sat, 30 Sep 2017 13:49:52 +0000 (15:49 +0200)
committerFabian Vogt <fabian@ritter-vogt.de>
Sat, 30 Sep 2017 14:28:47 +0000 (16:28 +0200)
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

src/kitemviews/kstandarditemlistgroupheader.cpp

index 2afaa168c5b6721aca3333c37019df1caa9dfa0f..c7997faf1c8c5e6e7af4919471d3d80798d93f39 100644 (file)
@@ -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);