From kde-commits Sat Sep 30 21:33:47 2017 From: Michael Pyne Date: Sat, 30 Sep 2017 21:33:47 +0000 To: kde-commits Subject: [juk/frameworks] /: Port juk.cpp away from KStandardDirs. Message-Id: X-MARC-Message: https://marc.info/?l=kde-commits&m=150680723827263 Git commit 2adf01550d9f41de06c47c7af8022fa9dddec3d5 by Michael Pyne. Committed on 30/09/2017 at 14:49. Pushed by mpyne into branch 'frameworks'. Port juk.cpp away from KStandardDirs. M +13 -13 juk.cpp https://commits.kde.org/juk/2adf01550d9f41de06c47c7af8022fa9dddec3d5 diff --git a/juk.cpp b/juk.cpp index b077df5..66797b6 100644 --- a/juk.cpp +++ b/juk.cpp @@ -19,13 +19,10 @@ = #include #include -#include #include #include #include #include -#include -#include #include #include #include @@ -33,9 +30,12 @@ #include #include = +#include +#include #include #include #include +#include #include #include #include @@ -73,11 +73,11 @@ void deleteAndClear(T *&ptr) = JuK::JuK(const QStringList &filesToOpen, QWidget *parent) : KXmlGuiWindow(parent, Qt::WindowFlags(Qt::WA_DeleteOnClose)), - m_splitter(0), - m_statusLabel(0), - m_systemTray(0), + m_splitter(nullptr), + m_statusLabel(nullptr), + m_systemTray(nullptr), m_player(new PlayerManager), - m_scrobbler(0), + m_scrobbler(nullptr), m_filesToOpen(filesToOpen), m_shuttingDown(false) { @@ -372,13 +372,13 @@ void JuK::slotClearOldCovers() // we find our tracks in a different order this run, which would cause= old saved // covers to be wrong. // See mpris2/mediaplayer2player.cpp - QStringList oldFiles =3D KGlobal::dirs()->findAllResources("tmp", "juk= -cover-*.png"); + QString tmpDir =3D QStandardPaths::writableLocation(QStandardPaths::Te= mpLocation); + QStringList nameFilters; = - foreach(const QString &file, oldFiles) { - qCWarning(JUK_LOG) << "Removing old cover" << file; - if(!QFile::remove(file)) { - qCCritical(JUK_LOG) << "Failed to remove old cover" << file; - } + nameFilters << QStringLiteral("juk-cover-*.png"); + QDirIterator jukCoverIter(tmpDir, nameFilters); + while (jukCoverIter.hasNext()) { + QFile::remove(jukCoverIter.next()); } } =