doc: Fix many small doxygen warnings

15 files changed, 171 insertions, 101 deletions

diff --git a/src/base/QXmppEntityTimeIq.cpp b/src/base/QXmppEntityTimeIq.cpp
index ce9e6d1b..7ba76000 100644
--- a/src/base/QXmppEntityTimeIq.cpp
+++ b/src/base/QXmppEntityTimeIq.cpp
@@ -28,47 +28,52 @@
 
 #include <QDomElement>
 
+///
 /// Returns the timezone offset in seconds.
 ///
-
 int QXmppEntityTimeIq::tzo() const
 {
     return m_tzo;
 }
 
+///
 /// Sets the timezone offset in seconds.
 ///
 /// \param tzo
-
+///
 void QXmppEntityTimeIq::setTzo(int tzo)
 {
     m_tzo = tzo;
 }
 
+///
 /// Returns the date/time in Coordinated Universal Time (UTC).
 ///
-
 QDateTime QXmppEntityTimeIq::utc() const
 {
     return m_utc;
 }
 
+///
 /// Sets the date/time in Coordinated Universal Time (UTC).
 ///
 /// \param utc
-
+///
 void QXmppEntityTimeIq::setUtc(const QDateTime &utc)
 {
     m_utc = utc;
 }
 
-/// \cond
+///
+/// Returns true, if the element is a valid entity time IQ.
+///
 bool QXmppEntityTimeIq::isEntityTimeIq(const QDomElement &element)
 {
     QDomElement timeElement = element.firstChildElement("time");
     return timeElement.namespaceURI() == ns_entity_time;
 }
 
+/// \cond
 void QXmppEntityTimeIq::parseElementFromChild(const QDomElement &element)
 {
     QDomElement timeElement = element.firstChildElement("time");
diff --git a/src/base/QXmppHttpUploadIq.cpp b/src/base/QXmppHttpUploadIq.cpp
index c2fbdda3..6b5e1df5 100644
--- a/src/base/QXmppHttpUploadIq.cpp
+++ b/src/base/QXmppHttpUploadIq.cpp
@@ -43,57 +43,68 @@ QXmppHttpUploadRequestIq::QXmppHttpUploadRequestIq()
 {
 }
 
+/// Default copy-constructor
 QXmppHttpUploadRequestIq::QXmppHttpUploadRequestIq(const QXmppHttpUploadRequestIq &) = default;
 
 QXmppHttpUploadRequestIq::~QXmppHttpUploadRequestIq() = default;
 
+/// Default assignment operator
 QXmppHttpUploadRequestIq &QXmppHttpUploadRequestIq::operator=(const QXmppHttpUploadRequestIq &) = default;
 
+///
 /// Returns the file name of the file to be uploaded.
-
+///
 QString QXmppHttpUploadRequestIq::fileName() const
 {
     return d->fileName;
 }
 
+///
 /// Sets the file name. The upload service will use this to create the upload/
 /// download URLs. This may also differ from the actual file name to get a
 /// different URL. It's not required to replace special characters (this is the
 /// server's job).
-
+///
 void QXmppHttpUploadRequestIq::setFileName(const QString &fileName)
 {
     d->fileName = fileName;
 }
 
+///
 /// Returns the file's size in bytes.
-
+///
 qint64 QXmppHttpUploadRequestIq::size() const
 {
     return d->size;
 }
 
+///
 /// Sets the file's size in bytes.
-
+///
 void QXmppHttpUploadRequestIq::setSize(qint64 size)
 {
     d->size = size;
 }
 
+///
 /// Returns the (optional) MIME-type of the file.
-
+///
 QMimeType QXmppHttpUploadRequestIq::contentType() const
 {
     return d->contentType;
 }
 
+///
 /// Sets the MIME-type of the file. This is optional.
-
+///
 void QXmppHttpUploadRequestIq::setContentType(const QMimeType &type)
 {
     d->contentType = type;
 }
 
+///
+/// Returns true, if the the element is an HTTP File Upload slot request IQ.
+///
 bool QXmppHttpUploadRequestIq::isHttpUploadRequestIq(const QDomElement &element)
 {
     if (element.tagName() == "iq") {
@@ -144,62 +155,75 @@ QXmppHttpUploadSlotIq::QXmppHttpUploadSlotIq()
 {
 }
 
+/// Default copy-constructor
 QXmppHttpUploadSlotIq::QXmppHttpUploadSlotIq(const QXmppHttpUploadSlotIq &) = default;
 
 QXmppHttpUploadSlotIq::~QXmppHttpUploadSlotIq() = default;
 
+/// Default assignment operator
 QXmppHttpUploadSlotIq &QXmppHttpUploadSlotIq::operator=(const QXmppHttpUploadSlotIq &) = default;
 
+///
 /// Returns the URL for uploading via. HTTP PUT.
-
+///
 QUrl QXmppHttpUploadSlotIq::putUrl() const
 {
     return d->putUrl;
 }
 
+///
 /// Sets the URL the client should use for uploading.
-
+///
 void QXmppHttpUploadSlotIq::setPutUrl(const QUrl &putUrl)
 {
     d->putUrl = putUrl;
 }
 
+///
 /// Returns the URL to where the file will be served.
-
+///
 QUrl QXmppHttpUploadSlotIq::getUrl() const
 {
     return d->getUrl;
 }
 
+///
 /// Sets the download URL.
-
+///
 void QXmppHttpUploadSlotIq::setGetUrl(const QUrl &getUrl)
 {
     d->getUrl = getUrl;
 }
 
+///
 /// Returns a map of header fields (header name -> value) that need to be
 /// included in the PUT (upload) request. This won't contain any other fields
 /// than: "Authorization", "Cookie" or "Expires".
-
+///
 QMap<QString, QString> QXmppHttpUploadSlotIq::putHeaders() const
 {
     return d->putHeaders;
 }
 
+///
 /// Sets the header fields the client needs to include in the PUT (upload)
 /// request. All fields other than "Authorization", "Cookie" or "Expires" will
 /// be ignored.
-
+///
 void QXmppHttpUploadSlotIq::setPutHeaders(const QMap<QString, QString> &putHeaders)
 {
     d->putHeaders.clear();
-    for (QString &name : putHeaders.keys()) {
-        if (name == "Authorization" || name == "Cookie" || name == "Expires")
-            d->putHeaders[name] = putHeaders[name];
-    }
+    std::for_each(putHeaders.keyBegin(), putHeaders.keyEnd(),
+                  [this, &putHeaders](const QString &name) {
+                      if (name == "Authorization" || name == "Cookie" || name == "Expires") {
+                          d->putHeaders.insert(name, putHeaders[name]);
+                      }
+                  });
 }
 
+///
+/// Returns true, if the the element is an HTTP File Upload slot result IQ.
+///
 bool QXmppHttpUploadSlotIq::isHttpUploadSlotIq(const QDomElement &element)
 {
     if (element.tagName() == "iq") {
@@ -236,14 +260,12 @@ void QXmppHttpUploadSlotIq::toXmlElementFromChild(QXmlStreamWriter *writer) cons
 
     writer->writeStartElement("put");
     writer->writeAttribute("url", d->putUrl.toEncoded());
-    if (!d->putHeaders.isEmpty()) {
-        for (const QString &name : d->putHeaders.keys()) {
-            writer->writeStartElement("header");
-            writer->writeAttribute("name", name);
-            writer->writeCharacters(d->putHeaders[name]);
-            writer->writeEndElement();
-        }
-    }
+    std::for_each(d->putHeaders.keyBegin(), d->putHeaders.keyEnd(), [=](const QString &name) {
+        writer->writeStartElement("header");
+        writer->writeAttribute("name", name);
+        writer->writeCharacters(d->putHeaders.value(name));
+        writer->writeEndElement();
+    });
     writer->writeEndElement();
 
     writer->writeStartElement("get");
diff --git a/src/base/QXmppMixIq.cpp b/src/base/QXmppMixIq.cpp
index e61616a7..32377d21 100644
--- a/src/base/QXmppMixIq.cpp
+++ b/src/base/QXmppMixIq.cpp
@@ -57,10 +57,12 @@ QXmppMixIq::QXmppMixIq()
 {
 }
 
+/// Default copy-constructor
 QXmppMixIq::QXmppMixIq(const QXmppMixIq&) = default;
 
 QXmppMixIq::~QXmppMixIq() = default;
 
+/// Default assignment operator
 QXmppMixIq& QXmppMixIq::operator=(const QXmppMixIq&) = default;
 
 /// Returns the channel JID. It also contains a participant id for Join/
diff --git a/src/base/QXmppMixItem.cpp b/src/base/QXmppMixItem.cpp
index 70d8bdd2..8006a27e 100644
--- a/src/base/QXmppMixItem.cpp
+++ b/src/base/QXmppMixItem.cpp
@@ -44,8 +44,10 @@ QXmppMixInfoItem::QXmppMixInfoItem()
 {
 }
 
+/// Default copy-constructor
 QXmppMixInfoItem::QXmppMixInfoItem(const QXmppMixInfoItem&) = default;
 
+/// Default assignment operator
 QXmppMixInfoItem& QXmppMixInfoItem::operator=(const QXmppMixInfoItem&) = default;
 
 QXmppMixInfoItem::~QXmppMixInfoItem() = default;
@@ -174,8 +176,10 @@ QXmppMixParticipantItem::QXmppMixParticipantItem()
 {
 }
 
+/// Default copy-constructor
 QXmppMixParticipantItem::QXmppMixParticipantItem(const QXmppMixParticipantItem&) = default;
 
+/// Default assignment operator
 QXmppMixParticipantItem& QXmppMixParticipantItem::operator=(const QXmppMixParticipantItem&) = default;
 
 QXmppMixParticipantItem::~QXmppMixParticipantItem() = default;
diff --git a/src/base/QXmppPingIq.cpp b/src/base/QXmppPingIq.cpp
index fab351a0..b2458aa7 100644
--- a/src/base/QXmppPingIq.cpp
+++ b/src/base/QXmppPingIq.cpp
@@ -28,10 +28,22 @@
 
 #include <QDomElement>
 
-QXmppPingIq::QXmppPingIq() : QXmppIq(QXmppIq::Get)
+///
+/// \class QXmppPingIq
+///
+/// QXmppPingIq represents a Ping IQ as defined by \xep{0199, XMPP Ping}.
+///
+/// \ingroup Stanzas
+///
+
+QXmppPingIq::QXmppPingIq()
+    : QXmppIq(QXmppIq::Get)
 {
 }
 
+///
+/// Returns true, if the element is a ping IQ.
+///
 bool QXmppPingIq::isPingIq(const QDomElement &element)
 {
     QDomElement pingElement = element.firstChildElement(QStringLiteral("ping"));
@@ -39,9 +51,11 @@ bool QXmppPingIq::isPingIq(const QDomElement &element)
             pingElement.namespaceURI() == ns_ping);
 }
 
+/// \cond
 void QXmppPingIq::toXmlElementFromChild(QXmlStreamWriter *writer) const
 {
     writer->writeStartElement(QStringLiteral("ping"));
     writer->writeDefaultNamespace(ns_ping);
     writer->writeEndElement();
 }
+/// \endcond
diff --git a/src/base/QXmppPingIq.h b/src/base/QXmppPingIq.h
index 93604301..e2582477 100644
--- a/src/base/QXmppPingIq.h
+++ b/src/base/QXmppPingIq.h
@@ -26,18 +26,15 @@
 
 #include "QXmppIq.h"
 
-///
-/// \brief QXmppPingIq represents a Ping IQ as defined by \xep{0199}: XMPP
-/// Ping.
-///
-/// \ingroup Stanzas
-///
 class QXMPP_EXPORT QXmppPingIq : public QXmppIq
 {
 public:
     QXmppPingIq();
-    void toXmlElementFromChild(QXmlStreamWriter *writer) const override;
     static bool isPingIq(const QDomElement &element);
+
+    /// \cond
+    void toXmlElementFromChild(QXmlStreamWriter *writer) const override;
+    /// \endcond
 };
 
 #endif
diff --git a/src/base/QXmppPubSubIq.cpp b/src/base/QXmppPubSubIq.cpp
index 98329647..ccde6412 100644
--- a/src/base/QXmppPubSubIq.cpp
+++ b/src/base/QXmppPubSubIq.cpp
@@ -64,10 +64,12 @@ QXmppPubSubIq::QXmppPubSubIq()
 {
 }
 
+/// Default copy-constructor
 QXmppPubSubIq::QXmppPubSubIq(const QXmppPubSubIq &iq) = default;
 
 QXmppPubSubIq::~QXmppPubSubIq() = default;
 
+/// Default assignment operator
 QXmppPubSubIq &QXmppPubSubIq::operator=(const QXmppPubSubIq &iq) = default;
 
 /// Returns the PubSub queryType for this IQ.
diff --git a/src/base/QXmppPubSubItem.cpp b/src/base/QXmppPubSubItem.cpp
index c55aca98..d136547c 100644
--- a/src/base/QXmppPubSubItem.cpp
+++ b/src/base/QXmppPubSubItem.cpp
@@ -40,10 +40,12 @@ QXmppPubSubItem::QXmppPubSubItem()
 {
 }
 
+/// Default copy-constructor
 QXmppPubSubItem::QXmppPubSubItem(const QXmppPubSubItem &iq) = default;
 
 QXmppPubSubItem::~QXmppPubSubItem() = default;
 
+/// Default assignment operator
 QXmppPubSubItem &QXmppPubSubItem::operator=(const QXmppPubSubItem &iq) = default;
 
 /// Returns the ID of the PubSub item.
diff --git a/src/base/QXmppPushEnableIq.cpp b/src/base/QXmppPushEnableIq.cpp
index c17f7d5e..80e88f9a 100644
--- a/src/base/QXmppPushEnableIq.cpp
+++ b/src/base/QXmppPushEnableIq.cpp
@@ -44,8 +44,12 @@ QXmppPushEnableIq::QXmppPushEnableIq()
 {
 }
 
+/// Default copy-constructor
 QXmppPushEnableIq::QXmppPushEnableIq(const QXmppPushEnableIq &) = default;
+
 QXmppPushEnableIq::~QXmppPushEnableIq() = default;
+
+/// Default assignment operator
 QXmppPushEnableIq &QXmppPushEnableIq::operator=(const QXmppPushEnableIq &) = default;
 
 ///
diff --git a/src/base/QXmppRegisterIq.cpp b/src/base/QXmppRegisterIq.cpp
index 18739382..27447f6d 100644
--- a/src/base/QXmppRegisterIq.cpp
+++ b/src/base/QXmppRegisterIq.cpp
@@ -59,10 +59,12 @@ QXmppRegisterIq::QXmppRegisterIq()
 {
 }
 
+/// Default copy-constructor
 QXmppRegisterIq::QXmppRegisterIq(const QXmppRegisterIq &other) = default;
 
 QXmppRegisterIq::~QXmppRegisterIq() = default;
 
+/// Default assignment operator
 QXmppRegisterIq &QXmppRegisterIq::operator=(const QXmppRegisterIq &other) = default;
 
 /// Constructs a regular change password request.
diff --git a/src/base/QXmppRosterIq.cpp b/src/base/QXmppRosterIq.cpp
index 6c4999c6..53993db6 100644
--- a/src/base/QXmppRosterIq.cpp
+++ b/src/base/QXmppRosterIq.cpp
@@ -46,10 +46,12 @@ QXmppRosterIq::QXmppRosterIq()
 {
 }
 
+/// Default copy-constructor
 QXmppRosterIq::QXmppRosterIq(const QXmppRosterIq &) = default;
 
 QXmppRosterIq::~QXmppRosterIq() = default;
 
+/// Default assignment operator
 QXmppRosterIq &QXmppRosterIq::operator=(const QXmppRosterIq &) = default;
 
 ///
@@ -191,10 +193,12 @@ QXmppRosterIq::Item::Item()
 {
 }
 
+/// Default copy-constructor
 QXmppRosterIq::Item::Item(const QXmppRosterIq::Item &other) = default;
 
 QXmppRosterIq::Item::~Item() = default;
 
+/// Default assignment operator
 QXmppRosterIq::Item &QXmppRosterIq::Item::operator=(const Item &other) = default;
 
 ///
diff --git a/src/base/QXmppStanza.cpp b/src/base/QXmppStanza.cpp
index 2fee0673..f087adcb 100644
--- a/src/base/QXmppStanza.cpp
+++ b/src/base/QXmppStanza.cpp
@@ -44,91 +44,91 @@ public:
     QString type;
 };
 
+///
 /// Constructs an empty extended address.
-
+///
 QXmppExtendedAddress::QXmppExtendedAddress()
     : d(new QXmppExtendedAddressPrivate())
 {
     d->delivered = false;
 }
 
-/// Constructs a copy of other.
-///
-/// \param other
-///
+/// Default copy-constructur
 QXmppExtendedAddress::QXmppExtendedAddress(const QXmppExtendedAddress &other) = default;
 
 QXmppExtendedAddress::~QXmppExtendedAddress() = default;
 
-/// Assigns the other address to this one.
-///
-/// \param other
-///
-QXmppExtendedAddress &QXmppExtendedAddress::operator=(const QXmppExtendedAddress &other)
-{
-    d = other.d;
-    return *this;
-}
+/// Default assignment operator
+QXmppExtendedAddress &QXmppExtendedAddress::operator=(const QXmppExtendedAddress &other) = default;
 
+///
 /// Returns the human-readable description of the address.
-
+///
 QString QXmppExtendedAddress::description() const
 {
     return d->description;
 }
 
+///
 /// Sets the human-readable \a description of the address.
-
+///
 void QXmppExtendedAddress::setDescription(const QString &description)
 {
     d->description = description;
 }
 
+///
 /// Returns the JID of the address.
-
+///
 QString QXmppExtendedAddress::jid() const
 {
     return d->jid;
 }
 
+///
 /// Sets the JID of the address.
-
+///
 void QXmppExtendedAddress::setJid(const QString &jid)
 {
     d->jid = jid;
 }
 
+///
 /// Returns the type of the address.
-
+///
 QString QXmppExtendedAddress::type() const
 {
     return d->type;
 }
 
+///
 /// Sets the \a type of the address.
-
+///
 void QXmppExtendedAddress::setType(const QString &type)
 {
     d->type = type;
 }
 
+///
 /// Returns whether the stanza has been delivered to this address.
-
+///
 bool QXmppExtendedAddress::isDelivered() const
 {
     return d->delivered;
 }
 
+///
 /// Sets whether the stanza has been \a delivered to this address.
-
+///
 void QXmppExtendedAddress::setDelivered(bool delivered)
 {
     d->delivered = delivered;
 }
 
+///
 /// Checks whether this address is valid. The extended address is considered
 /// to be valid if at least type and JID fields are non-empty.
-
+///
 bool QXmppExtendedAddress::isValid() const
 {
     return !d->type.isEmpty() && !d->jid.isEmpty();
@@ -350,59 +350,65 @@ void QXmppStanza::Error::setRedirectionUri(const QString &redirectionUri)
     d->redirectionUri = redirectionUri;
 }
 
+///
 /// Returns true, if an HTTP File Upload failed, because the file was too
 /// large.
 ///
 /// \since QXmpp 1.1
-
+///
 bool QXmppStanza::Error::fileTooLarge() const
 {
     return d->fileTooLarge;
 }
 
+///
 /// Sets whether the requested file for HTTP File Upload was too large.
 ///
 /// You should also set maxFileSize in this case.
 ///
 /// \since QXmpp 1.1
-
+///
 void QXmppStanza::Error::setFileTooLarge(bool fileTooLarge)
 {
     d->fileTooLarge = fileTooLarge;
 }
 
+///
 /// Returns the maximum file size allowed for uploading via. HTTP File Upload.
 ///
 /// \since QXmpp 1.1
-
+///
 qint64 QXmppStanza::Error::maxFileSize() const
 {
     return d->maxFileSize;
 }
 
+///
 /// Sets the maximum file size allowed for uploading via. HTTP File Upload.
 ///
 /// This sets fileTooLarge to true.
 ///
 /// \since QXmpp 1.1
-
+///
 void QXmppStanza::Error::setMaxFileSize(qint64 maxFileSize)
 {
     setFileTooLarge(true);
     d->maxFileSize = maxFileSize;
 }
 
+///
 /// Returns when to retry the upload request via. HTTP File Upload.
 ///
 /// \since QXmpp 1.1
-
+///
 QDateTime QXmppStanza::Error::retryDate() const
 {
     return d->retryDate;
 }
 
+///
 /// Sets the datetime when the client can retry to request the upload slot.
-
+///
 void QXmppStanza::Error::setRetryDate(const QDateTime &retryDate)
 {
     d->retryDate = retryDate;
@@ -514,11 +520,12 @@ public:
     QList<QXmppExtendedAddress> extendedAddresses;
 };
 
+///
 /// Constructs a QXmppStanza with the specified sender and recipient.
 ///
 /// \param from
 /// \param to
-
+///
 QXmppStanza::QXmppStanza(const QString &from, const QString &to)
     : d(new QXmppStanzaPrivate)
 {
@@ -527,136 +534,137 @@ QXmppStanza::QXmppStanza(const QString &from, const QString &to)
 }
 
 /// Constructs a copy of \a other.
-
-QXmppStanza::QXmppStanza(const QXmppStanza &other)
-    : d(other.d)
-{
-}
+QXmppStanza::QXmppStanza(const QXmppStanza &other) = default;
 
 /// Destroys a QXmppStanza.
-
-QXmppStanza::~QXmppStanza()
-{
-}
+QXmppStanza::~QXmppStanza() = default;
 
 /// Assigns \a other to this stanza.
+QXmppStanza &QXmppStanza::operator=(const QXmppStanza &other) = default;
 
-QXmppStanza &QXmppStanza::operator=(const QXmppStanza &other)
-{
-    d = other.d;
-    return *this;
-}
-
+///
 /// Returns the stanza's recipient JID.
 ///
-
 QString QXmppStanza::to() const
 {
     return d->to;
 }
 
+///
 /// Sets the stanza's recipient JID.
 ///
 /// \param to
-
+///
 void QXmppStanza::setTo(const QString &to)
 {
     d->to = to;
 }
 
+///
 /// Returns the stanza's sender JID.
-
+///
 QString QXmppStanza::from() const
 {
     return d->from;
 }
 
+///
 /// Sets the stanza's sender JID.
 ///
 /// \param from
-
+///
 void QXmppStanza::setFrom(const QString &from)
 {
     d->from = from;
 }
 
+///
 /// Returns the stanza's identifier.
-
+///
 QString QXmppStanza::id() const
 {
     return d->id;
 }
 
+///
 /// Sets the stanza's identifier.
 ///
 /// \param id
-
+///
 void QXmppStanza::setId(const QString &id)
 {
     d->id = id;
 }
 
+///
 /// Returns the stanza's language.
-
+///
 QString QXmppStanza::lang() const
 {
     return d->lang;
 }
 
+///
 /// Sets the stanza's language.
 ///
 /// \param lang
-
+///
 void QXmppStanza::setLang(const QString &lang)
 {
     d->lang = lang;
 }
 
+///
 /// Returns the stanza's error.
-
+///
 QXmppStanza::Error QXmppStanza::error() const
 {
     return d->error;
 }
 
+///
 /// Sets the stanza's error.
 ///
 /// \param error
-
+///
 void QXmppStanza::setError(const QXmppStanza::Error &error)
 {
     d->error = error;
 }
 
+///
 /// Returns the stanza's "extensions".
 ///
 /// Extensions are XML elements which are not handled internally by QXmpp.
-
+///
 QXmppElementList QXmppStanza::extensions() const
 {
     return d->extensions;
 }
 
+///
 /// Sets the stanza's "extensions".
 ///
 /// \param extensions
-
+///
 void QXmppStanza::setExtensions(const QXmppElementList &extensions)
 {
     d->extensions = extensions;
 }
 
-/// Returns the stanza's extended addresses as defined by
-/// \xep{0033}: Extended Stanza Addressing.
-
+///
+/// Returns the stanza's extended addresses as defined by \xep{0033, Extended
+/// Stanza Addressing}.
+///
 QList<QXmppExtendedAddress> QXmppStanza::extendedAddresses() const
 {
     return d->extendedAddresses;
 }
 
-/// Sets the stanza's extended addresses as defined by
-/// \xep{0033}: Extended Stanza Addressing.
-
+///
+/// Sets the stanza's extended addresses as defined by \xep{0033, Extended
+/// Stanza Addressing}.
+///
 void QXmppStanza::setExtendedAddresses(const QList<QXmppExtendedAddress> &addresses)
 {
     d->extendedAddresses = addresses;
diff --git a/src/client/QXmppMucManager.h b/src/client/QXmppMucManager.h
index de1ad5a8..74f404a3 100644
--- a/src/client/QXmppMucManager.h
+++ b/src/client/QXmppMucManager.h
@@ -59,6 +59,7 @@ class QXmppMucRoomPrivate;
 class QXMPP_EXPORT QXmppMucManager : public QXmppClientExtension
 {
     Q_OBJECT
+    /// List of joined MUC rooms
     Q_PROPERTY(QList<QXmppMucRoom *> rooms READ rooms NOTIFY roomAdded)
 
 public:
diff --git a/src/client/QXmppOutgoingClient.h b/src/client/QXmppOutgoingClient.h
index 9ad32545..d640d4e5 100644
--- a/src/client/QXmppOutgoingClient.h
+++ b/src/client/QXmppOutgoingClient.h
@@ -58,6 +58,7 @@ public:
     bool isStreamManagementEnabled() const;
     bool isStreamResumed() const;
 
+    /// Returns the used socket
     QSslSocket *socket() const { return QXmppStream::socket(); };
     QXmppStanza::Error::Condition xmppStreamError();
 
diff --git a/src/client/QXmppTransferManager.cpp b/src/client/QXmppTransferManager.cpp
index 84bbc1b1..7cf78b7f 100644
--- a/src/client/QXmppTransferManager.cpp
+++ b/src/client/QXmppTransferManager.cpp
@@ -464,7 +464,9 @@ void QXmppTransferJob::terminate(QXmppTransferJob::Error cause)
 
 /// \cond
 QXmppTransferIncomingJob::QXmppTransferIncomingJob(const QString &jid, QXmppClient *client, QObject *parent)
-    : QXmppTransferJob(jid, IncomingDirection, client, parent), m_candidateClient(nullptr), m_candidateTimer(nullptr)
+    : QXmppTransferJob(jid, IncomingDirection, client, parent),
+      m_candidateClient(nullptr),
+      m_candidateTimer(nullptr)
 {
 }