aboutsummaryrefslogtreecommitdiff
path: root/src/client/QXmppRosterManager.cpp
diff options
context:
space:
mode:
authorLinus Jahn <lnj@kaidan.im>2022-12-26 21:35:16 +0100
committerLinus Jahn <lnj@kaidan.im>2022-12-26 22:00:07 +0100
commit46995fd3fd7da4d1b035f71a1279c041b2871ffd (patch)
tree788517dbefa88fc1680b05a97e84f0e1d7403815 /src/client/QXmppRosterManager.cpp
parent71442ab927cd5c1c235528442fe3706f33e7ff6e (diff)
downloadqxmpp-46995fd3fd7da4d1b035f71a1279c041b2871ffd.tar.gz
Disable Qt keywords completely
Previously we had the policy that no qt keywords were allowed in headers that may be included by users. However since there was no automatic test verifying that in some places keywords were still used. This now disables qt keywords completely, also in tests and examples. Qt keywords are in general no good or really good idea as they even conflict with the standard library (`emit` at least). In some cases in the examples I just removed the slot tag if the functions didn't need to be slots (anymore). Closes #503.
Diffstat (limited to 'src/client/QXmppRosterManager.cpp')
-rw-r--r--src/client/QXmppRosterManager.cpp16
1 files changed, 8 insertions, 8 deletions
diff --git a/src/client/QXmppRosterManager.cpp b/src/client/QXmppRosterManager.cpp
index bb4eb435..810a4615 100644
--- a/src/client/QXmppRosterManager.cpp
+++ b/src/client/QXmppRosterManager.cpp
@@ -173,17 +173,17 @@ bool QXmppRosterManager::handleStanza(const QDomElement &element)
if (item.subscriptionType() == QXmppRosterIq::Item::Remove) {
if (d->entries.remove(bareJid)) {
// notify the user that the item was removed
- emit itemRemoved(bareJid);
+ Q_EMIT itemRemoved(bareJid);
}
} else {
const bool added = !d->entries.contains(bareJid);
d->entries.insert(bareJid, item);
if (added) {
// notify the user that the item was added
- emit itemAdded(bareJid);
+ Q_EMIT itemAdded(bareJid);
} else {
// notify the user that the item changed
- emit itemChanged(bareJid);
+ Q_EMIT itemChanged(bareJid);
}
}
}
@@ -196,7 +196,7 @@ bool QXmppRosterManager::handleStanza(const QDomElement &element)
}
if (isInitial) {
d->isRosterReceived = true;
- emit rosterReceived();
+ Q_EMIT rosterReceived();
}
break;
}
@@ -221,11 +221,11 @@ void QXmppRosterManager::_q_presenceReceived(const QXmppPresence &presence)
switch (presence.type()) {
case QXmppPresence::Available:
d->presences[bareJid][resource] = presence;
- emit presenceChanged(bareJid, resource);
+ Q_EMIT presenceChanged(bareJid, resource);
break;
case QXmppPresence::Unavailable:
d->presences[bareJid].remove(resource);
- emit presenceChanged(bareJid, resource);
+ Q_EMIT presenceChanged(bareJid, resource);
break;
case QXmppPresence::Subscribe:
if (client()->configuration().autoAcceptSubscriptions()) {
@@ -235,8 +235,8 @@ void QXmppRosterManager::_q_presenceReceived(const QXmppPresence &presence)
// ask for reciprocal subscription
subscribe(bareJid);
} else {
- emit subscriptionReceived(bareJid);
- emit subscriptionRequestReceived(bareJid, presence);
+ Q_EMIT subscriptionReceived(bareJid);
+ Q_EMIT subscriptionRequestReceived(bareJid, presence);
}
break;
default: