[SCM] kdeconnect packaging branch, master, updated. debian/0.9g-1-1183-g9d69498
Maximiliano Curia
maxy at moszumanska.debian.org
Fri Oct 14 14:29:53 UTC 2016
Gitweb-URL: http://git.debian.org/?p=pkg-kde/kde-extras/kdeconnect.git;a=commitdiff;h=e880e14
The following commit has been merged in the master branch:
commit e880e14ed55c15a9b1b9faee8c3e998cf4fb8dcd
Author: Albert Vaca <albertvaka at gmail.com>
Date: Tue Jun 21 14:50:17 2016 +0200
Preffer Q_FOREACH to foreach
---
cli/kdeconnect-cli.cpp | 2 +-
core/backends/lan/lanlinkprovider.cpp | 4 ++--
core/daemon.cpp | 10 +++++-----
core/device.cpp | 8 ++++----
core/pluginloader.cpp | 6 +++---
fileitemactionplugin/sendfileitemaction.cpp | 4 ++--
plugins/notifications/notificationsdbusinterface.cpp | 2 +-
plugins/runcommand/runcommand_config.cpp | 2 +-
plugins/sendnotifications/notificationslistener.cpp | 4 ++--
plugins/telepathy/telepathy-cm/connection.cpp | 12 ++++++------
tests/testnotificationlistener.cpp | 2 +-
11 files changed, 28 insertions(+), 28 deletions(-)
diff --git a/cli/kdeconnect-cli.cpp b/cli/kdeconnect-cli.cpp
index 6f386b6..2a9f95f 100644
--- a/cli/kdeconnect-cli.cpp
+++ b/cli/kdeconnect-cli.cpp
@@ -85,7 +85,7 @@ int main(int argc, char** argv)
reply.waitForFinished();
const QStringList devices = reply.value();
- foreach (const QString& id, devices) {
+ Q_FOREACH (const QString& id, devices) {
if (parser.isSet("id-only")) {
QTextStream(stdout) << id << endl;
} else {
diff --git a/core/backends/lan/lanlinkprovider.cpp b/core/backends/lan/lanlinkprovider.cpp
index b819f87..dce0251 100644
--- a/core/backends/lan/lanlinkprovider.cpp
+++ b/core/backends/lan/lanlinkprovider.cpp
@@ -293,7 +293,7 @@ void LanLinkProvider::sslErrors(const QList<QSslError>& errors)
disconnect(socket, SIGNAL(encrypted()), this, SLOT(encrypted()));
disconnect(socket, SIGNAL(sslErrors(QList<QSslError>)), this, SLOT(sslErrors(QList<QSslError>)));
- foreach(const QSslError &error, errors) {
+ Q_FOREACH (const QSslError &error, errors) {
qCDebug(KDECONNECT_CORE) << "SSL Error :" << error.errorString();
switch (error.error()) {
case QSslError::CertificateSignatureFailed:
@@ -322,7 +322,7 @@ void LanLinkProvider::sslErrors(const QList<QSslError>& errors)
void LanLinkProvider::sslErrorsLogButIgnore(const QList<QSslError>& errors)
{
- foreach(const QSslError &error, errors) {
+ Q_FOREACH (const QSslError &error, errors) {
qCDebug(KDECONNECT_CORE) << "SSL Error (ignoring):" << error.errorString();
}
}
diff --git a/core/daemon.cpp b/core/daemon.cpp
index b41616c..33b7619 100644
--- a/core/daemon.cpp
+++ b/core/daemon.cpp
@@ -69,7 +69,7 @@ Daemon::Daemon(QObject *parent, bool testMode)
//Read remebered paired devices
const QStringList& list = KdeConnectConfig::instance()->trustedDevices();
- Q_FOREACH(const QString& id, list) {
+ Q_FOREACH (const QString& id, list) {
Device* device = new Device(this, id);
connect(device, SIGNAL(reachableStatusChanged()), this, SLOT(onDeviceStatusChanged()));
connect(device, SIGNAL(trustedChanged(bool)), this, SLOT(onDeviceStatusChanged()));
@@ -122,7 +122,7 @@ void Daemon::removeDevice(Device* device)
void Daemon::cleanDevices()
{
- Q_FOREACH(Device* device, d->mDevices) {
+ Q_FOREACH (Device* device, d->mDevices) {
if (device->isTrusted()) {
continue;
}
@@ -145,7 +145,7 @@ void Daemon::forceOnNetworkChange()
// I hate this, but not able to find an alternative now
Device *Daemon::getDevice(QString deviceId) {
- Q_FOREACH(Device* device, d->mDevices) {
+ Q_FOREACH (Device* device, d->mDevices) {
if (device->id() == deviceId) {
return device;
}
@@ -156,7 +156,7 @@ Device *Daemon::getDevice(QString deviceId) {
QStringList Daemon::devices(bool onlyReachable, bool onlyTrusted) const
{
QStringList ret;
- Q_FOREACH(Device* device, d->mDevices) {
+ Q_FOREACH (Device* device, d->mDevices) {
if (onlyReachable && !device->isReachable()) continue;
if (onlyTrusted && !device->isTrusted()) continue;
ret.append(device->id());
@@ -243,7 +243,7 @@ bool Daemon::isDiscoveringDevices() const
QString Daemon::deviceIdByName(const QString &name) const
{
- foreach(Device* d, d->mDevices) {
+ Q_FOREACH (Device* d, d->mDevices) {
if (d->name() == name && d->isTrusted())
return d->id();
}
diff --git a/core/device.cpp b/core/device.cpp
index 74f5ff0..989c5de 100644
--- a/core/device.cpp
+++ b/core/device.cpp
@@ -113,7 +113,7 @@ void Device::reloadPlugins()
PluginLoader* loader = PluginLoader::instance();
const bool capabilitiesSupported = (m_protocolVersion >= 6);
- foreach (const QString& pluginName, loader->getPluginList()) {
+ Q_FOREACH (const QString& pluginName, loader->getPluginList()) {
const KPluginMetaData service = loader->getPluginInfo(pluginName);
const QSet<QString> incomingInterfaces = KPluginMetaData::readStringList(service.rawData(), "X-KdeConnect-SupportedPackageType").toSet();
const QSet<QString> outgoingInterfaces = KPluginMetaData::readStringList(service.rawData(), "X-KdeConnect-OutgoingPackageType").toSet();
@@ -149,10 +149,10 @@ void Device::reloadPlugins()
plugin = loader->instantiatePluginForDevice(pluginName, this);
}
- foreach(const QString& interface, incomingInterfaces) {
+ Q_FOREACH (const QString& interface, incomingInterfaces) {
newPluginsByIncomingInterface.insert(interface, plugin);
}
- foreach(const QString& interface, outgoingInterfaces) {
+ Q_FOREACH (const QString& interface, outgoingInterfaces) {
newPluginsByOutgoingInterface.insert(interface, plugin);
}
@@ -334,7 +334,7 @@ void Device::privateReceivedPackage(const NetworkPackage& np)
if (plugins.isEmpty()) {
qWarning() << "discarding unsupported package" << np.type() << "for" << name();
}
- foreach(KdeConnectPlugin* plugin, plugins) {
+ Q_FOREACH (KdeConnectPlugin* plugin, plugins) {
plugin->receivePackage(np);
}
} else {
diff --git a/core/pluginloader.cpp b/core/pluginloader.cpp
index b264598..0a55570 100644
--- a/core/pluginloader.cpp
+++ b/core/pluginloader.cpp
@@ -38,7 +38,7 @@ PluginLoader* PluginLoader::instance()
PluginLoader::PluginLoader()
{
QVector<KPluginMetaData> data = KPluginLoader::findPlugins("kdeconnect/");
- foreach (const KPluginMetaData& metadata, data) {
+ Q_FOREACH (const KPluginMetaData& metadata, data) {
plugins[metadata.pluginId()] = metadata;
}
}
@@ -87,7 +87,7 @@ KdeConnectPlugin* PluginLoader::instantiatePluginForDevice(const QString& plugin
QStringList PluginLoader::incomingInterfaces() const
{
QSet<QString> ret;
- foreach(const KPluginMetaData& service, plugins) {
+ Q_FOREACH (const KPluginMetaData& service, plugins) {
ret += KPluginMetaData::readStringList(service.rawData(), "X-KdeConnect-SupportedPackageType").toSet();
}
return ret.toList();
@@ -96,7 +96,7 @@ QStringList PluginLoader::incomingInterfaces() const
QStringList PluginLoader::outgoingInterfaces() const
{
QSet<QString> ret;
- foreach(const KPluginMetaData& service, plugins) {
+ Q_FOREACH (const KPluginMetaData& service, plugins) {
ret += KPluginMetaData::readStringList(service.rawData(), "X-KdeConnect-OutgoingPackageType").toSet();
}
return ret.toList();
diff --git a/fileitemactionplugin/sendfileitemaction.cpp b/fileitemactionplugin/sendfileitemaction.cpp
index 2b3b722..aff479c 100644
--- a/fileitemactionplugin/sendfileitemaction.cpp
+++ b/fileitemactionplugin/sendfileitemaction.cpp
@@ -55,7 +55,7 @@ QList<QAction*> SendFileItemAction::actions(const KFileItemListProperties& fileI
QDBusPendingReply<QStringList> reply = iface.devices(true, true);
reply.waitForFinished();
const QStringList devices = reply.value();
- foreach (const QString& id, devices) {
+ Q_FOREACH (const QString& id, devices) {
DeviceDbusInterface deviceIface(id);
if (!deviceIface.isValid()) {
continue;
@@ -89,7 +89,7 @@ void SendFileItemAction::sendFile()
{
QList<QUrl> urls = sender()->property("urls").value<QList<QUrl>>();
QString id = sender()->property("id").toString();
- foreach(const QUrl& url, urls) {
+ Q_FOREACH (const QUrl& url, urls) {
QDBusMessage msg = QDBusMessage::createMethodCall("org.kde.kdeconnect", "/modules/kdeconnect/devices/"+id+"/share", "org.kde.kdeconnect.device.share", "shareUrl");
msg.setArguments(QVariantList() << url.toString());
QDBusConnection::sessionBus().call(msg);
diff --git a/plugins/notifications/notificationsdbusinterface.cpp b/plugins/notifications/notificationsdbusinterface.cpp
index 471aae5..4b19955 100644
--- a/plugins/notifications/notificationsdbusinterface.cpp
+++ b/plugins/notifications/notificationsdbusinterface.cpp
@@ -69,7 +69,7 @@ void NotificationsDbusInterface::processPackage(const NetworkPackage& np)
id = id.mid(id.indexOf("::") + 2);
removeNotification(id);
} else if (np.get<bool>("isRequest")) {
- foreach (const auto& n, mNotifications) {
+ Q_FOREACH (const auto& n, mNotifications) {
NetworkPackage np(PACKAGE_TYPE_NOTIFICATION_REQUEST);
np.set("id", n->internalId());
np.set("appName", n->appName());
diff --git a/plugins/runcommand/runcommand_config.cpp b/plugins/runcommand/runcommand_config.cpp
index cc490e4..8f465bc 100644
--- a/plugins/runcommand/runcommand_config.cpp
+++ b/plugins/runcommand/runcommand_config.cpp
@@ -70,7 +70,7 @@ void RunCommandConfig::load()
QJsonDocument jsonDocument = QJsonDocument::fromJson(config()->get<QByteArray>("commands", "{}"));
QJsonObject jsonConfig = jsonDocument.object();
- foreach(const QString &key, jsonConfig.keys()) {
+ Q_FOREACH (const QString &key, jsonConfig.keys()) {
const QJsonObject entry = jsonConfig[key].toObject();
const QString name = entry["name"].toString();
const QString command = entry["command"].toString();
diff --git a/plugins/sendnotifications/notificationslistener.cpp b/plugins/sendnotifications/notificationslistener.cpp
index f907de6..fbd95b1 100644
--- a/plugins/sendnotifications/notificationslistener.cpp
+++ b/plugins/sendnotifications/notificationslistener.cpp
@@ -96,7 +96,7 @@ void NotificationsListener::loadApplications()
{
applications.clear();
QVariantList list = mPlugin->config()->getList("applications");
- foreach (const auto& a, list) {
+ Q_FOREACH (const auto& a, list) {
NotifyingApplication app = a.value<NotifyingApplication>();
if (!applications.contains(app.name))
applications.insert(app.name, app);
@@ -129,7 +129,7 @@ uint NotificationsListener::Notify(const QString &appName, uint replacesId,
applications.insert(app.name, app);
// update config:
QVariantList list;
- foreach (const auto& a, applications)
+ Q_FOREACH (const auto& a, applications)
list << QVariant::fromValue<NotifyingApplication>(a);
mPlugin->config()->setList("applications", list);
//qCDebug(KDECONNECT_PLUGIN_SENDNOTIFICATION) << "Added new application to config:" << app;
diff --git a/plugins/telepathy/telepathy-cm/connection.cpp b/plugins/telepathy/telepathy-cm/connection.cpp
index 22a5985..6a91cfc 100644
--- a/plugins/telepathy/telepathy-cm/connection.cpp
+++ b/plugins/telepathy/telepathy-cm/connection.cpp
@@ -127,7 +127,7 @@ QStringList ConnectConnection::inspectHandles(uint handleType, const Tp::UIntLis
QStringList result;
- foreach (uint handle, handles) {
+ Q_FOREACH (uint handle, handles) {
if (!m_handles.contains(handle)) {
return QStringList();
}
@@ -172,7 +172,7 @@ Tp::UIntList ConnectConnection::requestHandles(uint handleType, const QStringLis
return result;
}
- foreach(const QString &identify, identifiers) {
+ Q_FOREACH (const QString &identify, identifiers) {
uint handle = m_handles.key(identify, 0);
if (handle) {
result.append(handle);
@@ -188,7 +188,7 @@ Tp::ContactAttributesMap ConnectConnection::getContactListAttributes(const QStri
{
Tp::ContactAttributesMap contactAttributes;
- foreach (const uint handle, m_handles.keys()) {
+ Q_FOREACH (const uint handle, m_handles.keys()) {
if (handle == selfHandle()) {
continue;
}
@@ -209,7 +209,7 @@ Tp::ContactAttributesMap ConnectConnection::getContactAttributes(const Tp::UIntL
Tp::ContactAttributesMap contactAttributes;
- foreach (const uint handle, handles) {
+ Q_FOREACH (const uint handle, handles) {
if (m_handles.contains(handle)){
QVariantMap attributes;
attributes["org.freedesktop.Telepathy.Connection/contact-id"] = m_handles.value(handle);
@@ -253,7 +253,7 @@ uint ConnectConnection::addContacts(const QStringList &identifiers)
}
QList<uint> newHandles;
- foreach(const QString &identifier, identifiers) {
+ Q_FOREACH (const QString &identifier, identifiers) {
++handle;
m_handles.insert(handle, identifier);
newHandles << handle;
@@ -334,7 +334,7 @@ void ConnectConnection::setContactList(const QStringList &identifiers)
uint ConnectConnection::getHandle(const QString &identifier) const
{
- foreach (uint key, m_handles.keys()) {
+ Q_FOREACH (uint key, m_handles.keys()) {
if (m_handles.value(key) == identifier) {
return key;
}
diff --git a/tests/testnotificationlistener.cpp b/tests/testnotificationlistener.cpp
index 35ddedb..ad36dac 100644
--- a/tests/testnotificationlistener.cpp
+++ b/tests/testnotificationlistener.cpp
@@ -328,7 +328,7 @@ void TestNotificationListener::testNotify()
// icon synchronization:
int count = 0;
- foreach (const auto& iconName, KIconLoader::global()->queryIcons(-KIconLoader::SizeEnormous, KIconLoader::Application)) {
+ Q_FOREACH (const auto& iconName, KIconLoader::global()->queryIcons(-KIconLoader::SizeEnormous, KIconLoader::Application)) {
if (!iconName.endsWith(".png"))
continue;
if (count++ > 3) // max 3 iterations
--
kdeconnect packaging
More information about the pkg-kde-commits
mailing list