[SCM] ktp-kded-integration-module packaging branch, master, updated. debian/15.12.1-2-382-gbd961c2

Maximiliano Curia maxy at moszumanska.debian.org
Sat May 28 00:13:34 UTC 2016


Gitweb-URL: http://git.debian.org/?p=pkg-kde/applications/ktp-kded-module.git;a=commitdiff;h=cb62153

The following commit has been merged in the master branch:
commit cb62153277f8d85c6fb585f684d9ae63d63f7ee5
Author: Daniele E. Domenichelli <daniele.domenichelli at gmail.com>
Date:   Fri Dec 9 21:20:33 2011 +0100

    Use meaningful variable names instead of a,b and c
    
    Also mark unused variables as Q_UNUSED
---
 telepathy-mpris.cpp | 6 ++++--
 telepathy-mpris.h   | 2 +-
 2 files changed, 5 insertions(+), 3 deletions(-)

diff --git a/telepathy-mpris.cpp b/telepathy-mpris.cpp
index 22843e9..a8d5bee 100644
--- a/telepathy-mpris.cpp
+++ b/telepathy-mpris.cpp
@@ -177,9 +177,11 @@ void TelepathyMPRIS::onSettingsChanged()
     }
 }
 
-void TelepathyMPRIS::serviceOwnerChanged(const QString& a, const QString& b, const QString& c)
+void TelepathyMPRIS::serviceOwnerChanged(const QString &serviceName, const QString &oldOwner, const QString &newOwner)
 {
-    if (a.contains(QLatin1String("org.mpris.MediaPlayer2"))) {
+    Q_UNUSED(oldOwner)
+    Q_UNUSED(newOwner)
+    if (serviceName.contains(QLatin1String("org.mpris.MediaPlayer2"))) {
         kDebug() << "Found new mpris interface, running detection...";
         detectPlayers();
     }
diff --git a/telepathy-mpris.h b/telepathy-mpris.h
index 10bec22..11daf6b 100644
--- a/telepathy-mpris.h
+++ b/telepathy-mpris.h
@@ -39,7 +39,7 @@ public Q_SLOTS:
     void onPlayerSignalReceived(const QString &interface, const QVariantMap &changedProperties, const QStringList &invalidatedProperties);
     void onSettingsChanged();
     void detectPlayers();
-    void serviceOwnerChanged(const QString &a, const QString &b, const QString &c);
+    void serviceOwnerChanged(const QString &serviceName, const QString &oldOwner, const QString &newOwner);
     void onActivateNowPlaying();
     void onDeactivateNowPlaying();
 

-- 
ktp-kded-integration-module packaging



More information about the pkg-kde-commits mailing list