From 0710dee7e93ad0cfa0f0d12fdcefa7dc2d7c9865 Mon Sep 17 00:00:00 2001
From: Linus Jahn <lnj@kaidan.im>
Date: Mon, 20 Jul 2020 16:22:47 +0200
Subject: Clean up QXmppCarbonManager

---
 src/client/QXmppCarbonManager.cpp | 40 ++++++++++++++++++++++++++++-----------
 1 file changed, 29 insertions(+), 11 deletions(-)

(limited to 'src/client/QXmppCarbonManager.cpp')

diff --git a/src/client/QXmppCarbonManager.cpp b/src/client/QXmppCarbonManager.cpp
index e936ddf2..45dd96da 100644
--- a/src/client/QXmppCarbonManager.cpp
+++ b/src/client/QXmppCarbonManager.cpp
@@ -40,21 +40,24 @@ QXmppCarbonManager::~QXmppCarbonManager()
 {
 }
 
+///
 /// Returns whether message carbons are currently enabled
-
+///
 bool QXmppCarbonManager::carbonsEnabled() const
 {
     return m_carbonsEnabled;
 }
 
-/// Enable or disable message carbons.
+///
+/// Enables or disables message carbons for this connection.
+///
 /// This function does not check whether the server supports
 /// message carbons, but just sends the corresponding stanza
 /// to the server, so one must check in advance by using the
 /// discovery manager.
 ///
 /// By default, carbon copies are disabled.
-
+///
 void QXmppCarbonManager::setCarbonsEnabled(bool enabled)
 {
     if (m_carbonsEnabled == enabled)
@@ -73,6 +76,7 @@ void QXmppCarbonManager::setCarbonsEnabled(bool enabled)
     }
 }
 
+/// \cond
 QStringList QXmppCarbonManager::discoveryFeatures() const
 {
     return QStringList() << ns_carbons;
@@ -91,7 +95,7 @@ bool QXmppCarbonManager::handleStanza(const QDomElement &element)
     }
 
     if (carbon.isNull() || carbon.namespaceURI() != ns_carbons)
-        return false;  // Neither sent nor received -> no carbon message
+        return false;
 
     // carbon copies must always come from our bare JID
     if (element.attribute("from") != client()->configuration().jidBare()) {
@@ -99,16 +103,13 @@ bool QXmppCarbonManager::handleStanza(const QDomElement &element)
         return false;
     }
 
-    QDomElement forwarded = carbon.firstChildElement("forwarded");
-    if (forwarded.isNull())
-        return false;
-
-    QDomElement messageelement = forwarded.firstChildElement("message");
-    if (messageelement.isNull())
+    auto forwarded = carbon.firstChildElement("forwarded");
+    auto messageElement = forwarded.firstChildElement("message");
+    if (messageElement.isNull())
         return false;
 
     QXmppMessage message;
-    message.parse(messageelement);
+    message.parse(messageElement);
 
     if (sent)
         emit messageSent(message);
@@ -117,3 +118,20 @@ bool QXmppCarbonManager::handleStanza(const QDomElement &element)
 
     return true;
 }
+/// \endcond
+
+///
+/// \fn QXmppCarbonManager::messageReceived()
+///
+/// Emitted when a message was received from someone else and directed to
+/// another resource.
+///
+/// If you connect this signal to the QXmppClient::messageReceived signal, they
+/// will appear as normal messages.
+///
+
+///
+/// \fn QXmppCarbonManager::messageSent()
+///
+/// Emitted when another resource sent a message to someone else.
+///
-- 
cgit v1.2.3