modernize: Reformat ALL the source... again!
[quassel.git] / src / common / signalproxy.cpp
index a629551..9e47dd3 100644 (file)
 #include <QThread>
 
 #ifdef HAVE_SSL
-    #include <QSslSocket>
+#    include <QSslSocket>
 #endif
 
-#include "signalproxy.h"
-
 #include "peer.h"
 #include "protocol.h"
+#include "signalproxy.h"
 #include "syncableobject.h"
-#include "util.h"
 #include "types.h"
+#include "util.h"
 
 using namespace Protocol;
 
 class RemovePeerEvent : public QEvent
 {
 public:
-    RemovePeerEvent(Peer *peer) : QEvent(QEvent::Type(SignalProxy::RemovePeerEvent)), peer(peer) {}
-    Peer *peer;
+    RemovePeerEvent(Peer* peer)
+        : QEvent(QEvent::Type(SignalProxy::RemovePeerEvent))
+        , peer(peer)
+    {}
+    Peer* peer;
 };
 
-
 // ==================================================
 //  SignalRelay
 // ==================================================
 class SignalProxy::SignalRelay : public QObject
 {
-/* Q_OBJECT is not necessary or even allowed, because we implement
-   qt_metacall ourselves (and don't use any other features of the meta
-   object system)
-*/
+    /* Q_OBJECT is not necessary or even allowed, because we implement
+       qt_metacall ourselves (and don't use any other features of the meta
+       object system)
+    */
 public:
-    SignalRelay(SignalProxy *parent) : QObject(parent), _proxy(parent) {}
-    inline SignalProxy *proxy() const { return _proxy; }
+    SignalRelay(SignalProxy* parent)
+        : QObject(parent)
+        , _proxy(parent)
+    {}
+    inline SignalProxy* proxy() const { return _proxy; }
 
-    int qt_metacall(QMetaObject::Call _c, int _id, void **_a) override;
+    int qt_metacall(QMetaObject::Call _c, int _id, void** _a) override;
 
-    void attachSignal(QObject *sender, int signalId, const QByteArray &funcName);
-    void detachSignal(QObject *sender, int signalId = -1);
+    void attachSignal(QObject* sender, int signalId, const QByteArray& funcName);
+    void detachSignal(QObjectsender, int signalId = -1);
 
 private:
-    struct Signal {
-        QObject *sender{nullptr};
+    struct Signal
+    {
+        QObject* sender{nullptr};
         int signalId{-1};
         QByteArray signature;
-        Signal(QObject *sender, int sigId, QByteArray signature) : sender(sender), signalId(sigId), signature(std::move(signature)) {}
+        Signal(QObject* sender, int sigId, QByteArray signature)
+            : sender(sender)
+            , signalId(sigId)
+            , signature(std::move(signature))
+        {}
         Signal() = default;
     };
 
-    SignalProxy *_proxy;
+    SignalProxy_proxy;
     QHash<int, Signal> _slots;
 };
 
-
-void SignalProxy::SignalRelay::attachSignal(QObject *sender, int signalId, const QByteArray &funcName)
+void SignalProxy::SignalRelay::attachSignal(QObject* sender, int signalId, const QByteArray& funcName)
 {
     // we ride without safetybelts here... all checking for valid method etc pp has to be done by the caller
     // all connected methodIds are offset by the standard methodCount of QObject
@@ -106,8 +114,7 @@ void SignalProxy::SignalRelay::attachSignal(QObject *sender, int signalId, const
     QMetaObject::connect(sender, signalId, this, QObject::staticMetaObject.methodCount() + slotId);
 }
 
-
-void SignalProxy::SignalRelay::detachSignal(QObject *sender, int signalId)
+void SignalProxy::SignalRelay::detachSignal(QObject* sender, int signalId)
 {
     QHash<int, Signal>::iterator slotIter = _slots.begin();
     while (slotIter != _slots.end()) {
@@ -122,8 +129,7 @@ void SignalProxy::SignalRelay::detachSignal(QObject *sender, int signalId)
     }
 }
 
-
-int SignalProxy::SignalRelay::qt_metacall(QMetaObject::Call _c, int _id, void **_a)
+int SignalProxy::SignalRelay::qt_metacall(QMetaObject::Call _c, int _id, void** _a)
 {
     _id = QObject::qt_metacall(_c, _id, _a);
     if (_id < 0)
@@ -131,23 +137,26 @@ int SignalProxy::SignalRelay::qt_metacall(QMetaObject::Call _c, int _id, void **
 
     if (_c == QMetaObject::InvokeMetaMethod) {
         if (_slots.contains(_id)) {
-            QObject *caller = sender();
+            QObjectcaller = sender();
 
-            SignalProxy::ExtendedMetaObject *eMeta = proxy()->extendedMetaObject(caller->metaObject());
+            SignalProxy::ExtendedMetaObjecteMeta = proxy()->extendedMetaObject(caller->metaObject());
             Q_ASSERT(eMeta);
 
-            const Signal &signal = _slots[_id];
+            const Signalsignal = _slots[_id];
 
             QVariantList params;
 
-            const QList<int> &argTypes = eMeta->argTypes(signal.signalId);
+            const QList<int>argTypes = eMeta->argTypes(signal.signalId);
             for (int i = 0; i < argTypes.size(); i++) {
                 if (argTypes[i] == 0) {
-                    qWarning() << "SignalRelay::qt_metacall(): received invalid data for argument number" << i << "of signal" << QString("%1::%2").arg(caller->metaObject()->className()).arg(caller->metaObject()->method(signal.signalId).methodSignature().constData());
+                    qWarning() << "SignalRelay::qt_metacall(): received invalid data for argument number" << i << "of signal"
+                               << QString("%1::%2")
+                                      .arg(caller->metaObject()->className())
+                                      .arg(caller->metaObject()->method(signal.signalId).methodSignature().constData());
                     qWarning() << "                            - make sure all your data types are known by the Qt MetaSystem";
                     return _id;
                 }
-                params << QVariant(argTypes[i], _a[i+1]);
+                params << QVariant(argTypes[i], _a[i + 1]);
             }
 
             if (proxy()->_restrictMessageTarget) {
@@ -155,7 +164,8 @@ int SignalProxy::SignalRelay::qt_metacall(QMetaObject::Call _c, int _id, void **
                     if (peer != nullptr)
                         proxy()->dispatch(peer, RpcCall(signal.signature, params));
                 }
-            } else
+            }
+            else
                 proxy()->dispatch(RpcCall(signal.signature, params));
         }
         _id -= _slots.count();
@@ -163,38 +173,35 @@ int SignalProxy::SignalRelay::qt_metacall(QMetaObject::Call _c, int _id, void **
     return _id;
 }
 
-
 // ==================================================
 //  SignalProxy
 // ==================================================
 
 namespace {
-thread_local SignalProxy *_current{nullptr};
+thread_local SignalProxy_current{nullptr};
 }
 
-SignalProxy::SignalProxy(QObject *parent)
+SignalProxy::SignalProxy(QObjectparent)
     : QObject(parent)
 {
     setProxyMode(Client);
     init();
 }
 
-
-SignalProxy::SignalProxy(ProxyMode mode, QObject *parent)
+SignalProxy::SignalProxy(ProxyMode mode, QObject* parent)
     : QObject(parent)
 {
     setProxyMode(mode);
     init();
 }
 
-
 SignalProxy::~SignalProxy()
 {
     QHash<QByteArray, ObjectId>::iterator classIter = _syncSlave.begin();
     while (classIter != _syncSlave.end()) {
         ObjectId::iterator objIter = classIter->begin();
         while (objIter != classIter->end()) {
-            SyncableObject *obj = objIter.value();
+            SyncableObjectobj = objIter.value();
             objIter = classIter->erase(objIter);
             obj->stopSynchronize(this);
         }
@@ -207,13 +214,11 @@ SignalProxy::~SignalProxy()
     _current = nullptr;
 }
 
-
-SignalProxy *SignalProxy::current()
+SignalProxy* SignalProxy::current()
 {
     return _current;
 }
 
-
 void SignalProxy::setProxyMode(ProxyMode mode)
 {
     if (!_peerMap.empty()) {
@@ -240,18 +245,13 @@ void SignalProxy::init()
     updateSecureState();
 }
 
-
-void SignalProxy::initServer()
-{
-}
-
+void SignalProxy::initServer() {}
 
 void SignalProxy::initClient()
 {
-    attachSlot("__objectRenamed__", this, SLOT(objectRenamed(QByteArray,QString,QString)));
+    attachSlot("__objectRenamed__", this, SLOT(objectRenamed(QByteArray, QString, QString)));
 }
 
-
 void SignalProxy::setHeartBeatInterval(int secs)
 {
     if (_heartBeatInterval != secs) {
@@ -260,7 +260,6 @@ void SignalProxy::setHeartBeatInterval(int secs)
     }
 }
 
-
 void SignalProxy::setMaxHeartBeatCount(int max)
 {
     if (_maxHeartBeatCount != max) {
@@ -269,8 +268,7 @@ void SignalProxy::setMaxHeartBeatCount(int max)
     }
 }
 
-
-bool SignalProxy::addPeer(Peer *peer)
+bool SignalProxy::addPeer(Peer* peer)
 {
     if (!peer)
         return false;
@@ -313,19 +311,17 @@ bool SignalProxy::addPeer(Peer *peer)
     return true;
 }
 
-
 void SignalProxy::removeAllPeers()
 {
     Q_ASSERT(proxyMode() == Server || peerCount() <= 1);
     // wee need to copy that list since we modify it in the loop
-    QList<Peer *> peers = _peerMap.values();
+    QList<Peer*> peers = _peerMap.values();
     for (auto peer : peers) {
         removePeer(peer);
     }
 }
 
-
-void SignalProxy::removePeer(Peer *peer)
+void SignalProxy::removePeer(Peer* peer)
 {
     if (!peer) {
         qWarning() << Q_FUNC_INFO << "Trying to remove a null peer!";
@@ -357,45 +353,40 @@ void SignalProxy::removePeer(Peer *peer)
         emit disconnected();
 }
 
-
 void SignalProxy::removePeerBySender()
 {
-    removePeer(qobject_cast<Peer *>(sender()));
+    removePeer(qobject_cast<Peer*>(sender()));
 }
 
-
-void SignalProxy::renameObject(const SyncableObject *obj, const QString &newname, const QString &oldname)
+void SignalProxy::renameObject(const SyncableObject* obj, const QString& newname, const QString& oldname)
 {
     if (proxyMode() == Client)
         return;
 
-    const QMetaObject *meta = obj->syncMetaObject();
+    const QMetaObjectmeta = obj->syncMetaObject();
     const QByteArray className(meta->className());
     objectRenamed(className, newname, oldname);
 
     dispatch(RpcCall("__objectRenamed__", QVariantList() << className << newname << oldname));
 }
 
-
-void SignalProxy::objectRenamed(const QByteArray &classname, const QString &newname, const QString &oldname)
+void SignalProxy::objectRenamed(const QByteArray& classname, const QString& newname, const QString& oldname)
 {
     if (_syncSlave.contains(classname) && _syncSlave[classname].contains(oldname) && oldname != newname) {
-        SyncableObject *obj = _syncSlave[classname][newname] = _syncSlave[classname].take(oldname);
+        SyncableObjectobj = _syncSlave[classname][newname] = _syncSlave[classname].take(oldname);
         requestInit(obj);
     }
 }
 
-
-const QMetaObject *SignalProxy::metaObject(const QObject *obj)
+const QMetaObject* SignalProxy::metaObject(const QObject* obj)
 {
-    if (const auto *syncObject = qobject_cast<const SyncableObject *>(obj))
+    if (const auto* syncObject = qobject_cast<const SyncableObject*>(obj))
         return syncObject->syncMetaObject();
     else
         return obj->metaObject();
 }
 
-
-SignalProxy::ExtendedMetaObject *SignalProxy::extendedMetaObject(const QMetaObject *meta) const
+SignalProxy::ExtendedMetaObject* SignalProxy::extendedMetaObject(const QMetaObject* meta) const
 {
     if (_extendedMetaObjects.contains(meta))
         return _extendedMetaObjects[meta];
@@ -403,8 +394,7 @@ SignalProxy::ExtendedMetaObject *SignalProxy::extendedMetaObject(const QMetaObje
         return nullptr;
 }
 
-
-SignalProxy::ExtendedMetaObject *SignalProxy::createExtendedMetaObject(const QMetaObject *meta, bool checkConflicts)
+SignalProxy::ExtendedMetaObject* SignalProxy::createExtendedMetaObject(const QMetaObject* meta, bool checkConflicts)
 {
     if (!_extendedMetaObjects.contains(meta)) {
         _extendedMetaObjects[meta] = new ExtendedMetaObject(meta, checkConflicts);
@@ -412,10 +402,9 @@ SignalProxy::ExtendedMetaObject *SignalProxy::createExtendedMetaObject(const QMe
     return _extendedMetaObjects[meta];
 }
 
-
-bool SignalProxy::attachSignal(QObject *sender, const char *signal, const QByteArray &sigName)
+bool SignalProxy::attachSignal(QObject* sender, const char* signal, const QByteArray& sigName)
 {
-    const QMetaObject *meta = sender->metaObject();
+    const QMetaObjectmeta = sender->metaObject();
     QByteArray sig(meta->normalizedSignature(signal).mid(1));
     int methodId = meta->indexOfMethod(sig.constData());
     if (methodId == -1 || meta->method(methodId).methodType() != QMetaMethod::Signal) {
@@ -431,10 +420,9 @@ bool SignalProxy::attachSignal(QObject *sender, const char *signal, const QByteA
     return true;
 }
 
-
-bool SignalProxy::attachSlot(const QByteArray &sigName, QObject *recv, const char *slot)
+bool SignalProxy::attachSlot(const QByteArray& sigName, QObject* recv, const char* slot)
 {
-    const QMetaObject *meta = recv->metaObject();
+    const QMetaObjectmeta = recv->metaObject();
     int methodId = meta->indexOfMethod(meta->normalizedSignature(slot).mid(1));
     if (methodId == -1 || meta->method(methodId).methodType() == QMetaMethod::Method) {
         qWarning() << "SignalProxy::attachSlot(): No such slot" << slot;
@@ -451,8 +439,7 @@ bool SignalProxy::attachSlot(const QByteArray &sigName, QObject *recv, const cha
     return true;
 }
 
-
-void SignalProxy::synchronize(SyncableObject *obj)
+void SignalProxy::synchronize(SyncableObject* obj)
 {
     createExtendedMetaObject(obj, true);
 
@@ -474,8 +461,7 @@ void SignalProxy::synchronize(SyncableObject *obj)
     obj->synchronize(this);
 }
 
-
-void SignalProxy::detachObject(QObject *obj)
+void SignalProxy::detachObject(QObject* obj)
 {
     // Don't try to connect SignalProxy from itself on shutdown
     if (obj != this) {
@@ -484,14 +470,12 @@ void SignalProxy::detachObject(QObject *obj)
     }
 }
 
-
-void SignalProxy::detachSignals(QObject *sender)
+void SignalProxy::detachSignals(QObject* sender)
 {
     _signalRelay->detachSignal(sender);
 }
 
-
-void SignalProxy::detachSlots(QObject *receiver)
+void SignalProxy::detachSlots(QObject* receiver)
 {
     SlotHash::iterator slotIter = _attachedSlots.begin();
     while (slotIter != _attachedSlots.end()) {
@@ -503,8 +487,7 @@ void SignalProxy::detachSlots(QObject *receiver)
     }
 }
 
-
-void SignalProxy::stopSynchronize(SyncableObject *obj)
+void SignalProxy::stopSynchronize(SyncableObject* obj)
 {
     // we can't use a className here, since it might be effed up, if we receive the call as a result of a decon
     // gladly the objectName() is still valid. So we have only to iterate over the classes not each instance! *sigh*
@@ -519,18 +502,16 @@ void SignalProxy::stopSynchronize(SyncableObject *obj)
     obj->stopSynchronize(this);
 }
 
-
 template<class T>
-void SignalProxy::dispatch(const T &protoMessage)
+void SignalProxy::dispatch(const TprotoMessage)
 {
     for (auto&& peer : _peerMap.values()) {
         dispatch(peer, protoMessage);
     }
 }
 
-
 template<class T>
-void SignalProxy::dispatch(Peer *peer, const T &protoMessage)
+void SignalProxy::dispatch(Peer* peer, const T& protoMessage)
 {
     _targetPeer = peer;
 
@@ -542,19 +523,20 @@ void SignalProxy::dispatch(Peer *peer, const T &protoMessage)
     _targetPeer = nullptr;
 }
 
-
-void SignalProxy::handle(Peer *peer, const SyncMessage &syncMessage)
+void SignalProxy::handle(Peer* peer, const SyncMessage& syncMessage)
 {
     if (!_syncSlave.contains(syncMessage.className) || !_syncSlave[syncMessage.className].contains(syncMessage.objectName)) {
-        qWarning() << QString("no registered receiver for sync call: %1::%2 (objectName=\"%3\"). Params are:").arg(syncMessage.className, syncMessage.slotName, syncMessage.objectName)
+        qWarning() << QString("no registered receiver for sync call: %1::%2 (objectName=\"%3\"). Params are:")
+                          .arg(syncMessage.className, syncMessage.slotName, syncMessage.objectName)
                    << syncMessage.params;
         return;
     }
 
-    SyncableObject *receiver = _syncSlave[syncMessage.className][syncMessage.objectName];
-    ExtendedMetaObject *eMeta = extendedMetaObject(receiver);
+    SyncableObjectreceiver = _syncSlave[syncMessage.className][syncMessage.objectName];
+    ExtendedMetaObjecteMeta = extendedMetaObject(receiver);
     if (!eMeta->slotMap().contains(syncMessage.slotName)) {
-        qWarning() << QString("no matching slot for sync call: %1::%2 (objectName=\"%3\"). Params are:").arg(syncMessage.className, syncMessage.slotName, syncMessage.objectName)
+        qWarning() << QString("no matching slot for sync call: %1::%2 (objectName=\"%3\"). Params are:")
+                          .arg(syncMessage.className, syncMessage.slotName, syncMessage.objectName)
                    << syncMessage.params;
         return;
     }
@@ -591,87 +573,89 @@ void SignalProxy::handle(Peer *peer, const SyncMessage &syncMessage)
     invokeSlot(receiver, eMeta->updatedRemotelyId());
 }
 
-
-void SignalProxy::handle(Peer *peer, const InitRequest &initRequest)
+void SignalProxy::handle(Peer* peer, const InitRequest& initRequest)
 {
-   if (!_syncSlave.contains(initRequest.className)) {
-        qWarning() << "SignalProxy::handleInitRequest() received initRequest for unregistered Class:"
-                   << initRequest.className;
+    if (!_syncSlave.contains(initRequest.className)) {
+        qWarning() << "SignalProxy::handleInitRequest() received initRequest for unregistered Class:" << initRequest.className;
         return;
     }
 
     if (!_syncSlave[initRequest.className].contains(initRequest.objectName)) {
-        qWarning() << "SignalProxy::handleInitRequest() received initRequest for unregistered Object:"
-                   << initRequest.className << initRequest.objectName;
+        qWarning() << "SignalProxy::handleInitRequest() received initRequest for unregistered Object:" << initRequest.className
+                   << initRequest.objectName;
         return;
     }
 
-    SyncableObject *obj = _syncSlave[initRequest.className][initRequest.objectName];
+    SyncableObjectobj = _syncSlave[initRequest.className][initRequest.objectName];
     _targetPeer = peer;
     peer->dispatch(InitData(initRequest.className, initRequest.objectName, initData(obj)));
     _targetPeer = nullptr;
 }
 
-
-void SignalProxy::handle(Peer *peer, const InitData &initData)
+void SignalProxy::handle(Peer* peer, const InitData& initData)
 {
     Q_UNUSED(peer)
 
     if (!_syncSlave.contains(initData.className)) {
-        qWarning() << "SignalProxy::handleInitData() received initData for unregistered Class:"
-                   << initData.className;
+        qWarning() << "SignalProxy::handleInitData() received initData for unregistered Class:" << initData.className;
         return;
     }
 
     if (!_syncSlave[initData.className].contains(initData.objectName)) {
-        qWarning() << "SignalProxy::handleInitData() received initData for unregistered Object:"
-                   << initData.className << initData.objectName;
+        qWarning() << "SignalProxy::handleInitData() received initData for unregistered Object:" << initData.className << initData.objectName;
         return;
     }
 
-    SyncableObject *obj = _syncSlave[initData.className][initData.objectName];
+    SyncableObjectobj = _syncSlave[initData.className][initData.objectName];
     setInitData(obj, initData.initData);
 }
 
-
-void SignalProxy::handle(Peer *peer, const RpcCall &rpcCall)
+void SignalProxy::handle(Peer* peer, const RpcCall& rpcCall)
 {
-    QObject *receiver;
+    QObjectreceiver;
     int methodId;
     SlotHash::const_iterator slot = _attachedSlots.constFind(rpcCall.slotName);
     while (slot != _attachedSlots.constEnd() && slot.key() == rpcCall.slotName) {
         receiver = (*slot).first;
         methodId = (*slot).second;
         if (!invokeSlot(receiver, methodId, rpcCall.params, peer)) {
-            ExtendedMetaObject *eMeta = extendedMetaObject(receiver);
+            ExtendedMetaObjecteMeta = extendedMetaObject(receiver);
             qWarning("SignalProxy::handleSignal(): invokeMethod for \"%s\" failed ", eMeta->methodName(methodId).constData());
         }
         ++slot;
     }
 }
 
-
-bool SignalProxy::invokeSlot(QObject *receiver, int methodId, const QVariantList &params, QVariant &returnValue, Peer *peer)
+bool SignalProxy::invokeSlot(QObject* receiver, int methodId, const QVariantList& params, QVariant& returnValue, Peer* peer)
 {
-    ExtendedMetaObject *eMeta = extendedMetaObject(receiver);
+    ExtendedMetaObjecteMeta = extendedMetaObject(receiver);
     const QList<int> args = eMeta->argTypes(methodId);
-    const int numArgs = params.count() < args.count()
-                        ? params.count()
-                        : args.count();
+    const int numArgs = params.count() < args.count() ? params.count() : args.count();
 
     if (eMeta->minArgCount(methodId) > params.count()) {
         qWarning() << "SignalProxy::invokeSlot(): not enough params to invoke" << eMeta->methodName(methodId);
         return false;
     }
 
-    void *_a[] = { nullptr,           // return type...
-                   nullptr, nullptr, nullptr, nullptr, nullptr, // and 10 args - that's the max size qt can handle with signals and slots
-                   nullptr, nullptr, nullptr, nullptr, nullptr };
+    void* _a[] = {nullptr,  // return type...
+                  nullptr,
+                  nullptr,
+                  nullptr,
+                  nullptr,
+                  nullptr,  // and 10 args - that's the max size qt can handle with signals and slots
+                  nullptr,
+                  nullptr,
+                  nullptr,
+                  nullptr,
+                  nullptr};
 
     // check for argument compatibility and build params array
     for (int i = 0; i < numArgs; i++) {
         if (!params[i].isValid()) {
-            qWarning() << "SignalProxy::invokeSlot(): received invalid data for argument number" << i << "of method" << QString("%1::%2()").arg(receiver->metaObject()->className()).arg(receiver->metaObject()->method(methodId).methodSignature().constData());
+            qWarning() << "SignalProxy::invokeSlot(): received invalid data for argument number" << i << "of method"
+                       << QString("%1::%2()")
+                              .arg(receiver->metaObject()->className())
+                              .arg(receiver->metaObject()->method(methodId).methodSignature().constData());
             qWarning() << "                            - make sure all your data types are known by the Qt MetaSystem";
             return false;
         }
@@ -680,37 +664,34 @@ bool SignalProxy::invokeSlot(QObject *receiver, int methodId, const QVariantList
             return false;
         }
 
-        _a[i+1] = const_cast<void *>(params[i].constData());
+        _a[i + 1] = const_cast<void*>(params[i].constData());
     }
 
     if (returnValue.type() != QVariant::Invalid)
-        _a[0] = const_cast<void *>(returnValue.constData());
+        _a[0] = const_cast<void*>(returnValue.constData());
 
-    Qt::ConnectionType type = QThread::currentThread() == receiver->thread()
-                              ? Qt::DirectConnection
-                              : Qt::QueuedConnection;
+    Qt::ConnectionType type = QThread::currentThread() == receiver->thread() ? Qt::DirectConnection : Qt::QueuedConnection;
 
     if (type == Qt::DirectConnection) {
         _sourcePeer = peer;
         auto result = receiver->qt_metacall(QMetaObject::InvokeMetaMethod, methodId, _a) < 0;
         _sourcePeer = nullptr;
         return result;
-    } else {
+    }
+    else {
         qWarning() << "Queued Connections are not implemented yet";
         // note to self: qmetaobject.cpp:990 ff
         return false;
     }
 }
 
-
-bool SignalProxy::invokeSlot(QObject *receiver, int methodId, const QVariantList &params, Peer *peer)
+bool SignalProxy::invokeSlot(QObject* receiver, int methodId, const QVariantList& params, Peer* peer)
 {
     QVariant ret;
     return invokeSlot(receiver, methodId, params, ret, peer);
 }
 
-
-void SignalProxy::requestInit(SyncableObject *obj)
+void SignalProxy::requestInit(SyncableObject* obj)
 {
     if (proxyMode() == Server || obj->isInitialized())
         return;
@@ -718,14 +699,12 @@ void SignalProxy::requestInit(SyncableObject *obj)
     dispatch(InitRequest(obj->syncMetaObject()->className(), obj->objectName()));
 }
 
-
-QVariantMap SignalProxy::initData(SyncableObject *obj) const
+QVariantMap SignalProxy::initData(SyncableObject* obj) const
 {
     return obj->toVariantMap();
 }
 
-
-void SignalProxy::setInitData(SyncableObject *obj, const QVariantMap &properties)
+void SignalProxy::setInitData(SyncableObject* obj, const QVariantMap& properties)
 {
     if (obj->isInitialized())
         return;
@@ -735,12 +714,11 @@ void SignalProxy::setInitData(SyncableObject *obj, const QVariantMap &properties
     invokeSlot(obj, extendedMetaObject(obj)->updatedRemotelyId());
 }
 
-
-void SignalProxy::customEvent(QEvent *event)
+void SignalProxy::customEvent(QEvent* event)
 {
     switch ((int)event->type()) {
     case RemovePeerEvent: {
-        auto *e = static_cast< ::RemovePeerEvent *>(event);
+        auto* e = static_cast<::RemovePeerEvent*>(event);
         removePeer(e->peer);
         event->accept();
         break;
@@ -752,26 +730,26 @@ void SignalProxy::customEvent(QEvent *event)
     }
 }
 
-
-void SignalProxy::sync_call__(const SyncableObject *obj, SignalProxy::ProxyMode modeType, const char *funcname, va_list ap)
+void SignalProxy::sync_call__(const SyncableObject* obj, SignalProxy::ProxyMode modeType, const char* funcname, va_list ap)
 {
     // qDebug() << obj << modeType << "(" << _proxyMode << ")" << funcname;
     if (modeType != _proxyMode)
         return;
 
-    ExtendedMetaObject *eMeta = extendedMetaObject(obj);
+    ExtendedMetaObjecteMeta = extendedMetaObject(obj);
 
     QVariantList params;
 
-    const QList<int> &argTypes = eMeta->argTypes(eMeta->methodId(QByteArray(funcname)));
+    const QList<int>argTypes = eMeta->argTypes(eMeta->methodId(QByteArray(funcname)));
 
     for (int i = 0; i < argTypes.size(); i++) {
         if (argTypes[i] == 0) {
-            qWarning() << Q_FUNC_INFO << "received invalid data for argument number" << i << "of signal" << QString("%1::%2").arg(eMeta->metaObject()->className()).arg(funcname);
+            qWarning() << Q_FUNC_INFO << "received invalid data for argument number" << i << "of signal"
+                       << QString("%1::%2").arg(eMeta->metaObject()->className()).arg(funcname);
             qWarning() << "        - make sure all your data types are known by the Qt MetaSystem";
             return;
         }
-        params << QVariant(argTypes[i], va_arg(ap, void *));
+        params << QVariant(argTypes[i], va_arg(ap, void*));
     }
 
     if (_restrictMessageTarget) {
@@ -779,22 +757,21 @@ void SignalProxy::sync_call__(const SyncableObject *obj, SignalProxy::ProxyMode
             if (peer != nullptr)
                 dispatch(peer, SyncMessage(eMeta->metaObject()->className(), obj->objectName(), QByteArray(funcname), params));
         }
-    } else
+    }
+    else
         dispatch(SyncMessage(eMeta->metaObject()->className(), obj->objectName(), QByteArray(funcname), params));
 }
 
-
-void SignalProxy::disconnectDevice(QIODevice *dev, const QString &reason)
+void SignalProxy::disconnectDevice(QIODevice* dev, const QString& reason)
 {
     if (!reason.isEmpty())
         qWarning() << qPrintable(reason);
-    auto *sock  = qobject_cast<QAbstractSocket *>(dev);
+    auto* sock = qobject_cast<QAbstractSocket*>(dev);
     if (sock)
         qWarning() << qPrintable(tr("Disconnecting")) << qPrintable(sock->peerAddress().toString());
     dev->close();
 }
 
-
 void SignalProxy::dumpProxyStats()
 {
     QString mode;
@@ -804,8 +781,8 @@ void SignalProxy::dumpProxyStats()
         mode = "Client";
 
     int slaveCount = 0;
-    foreach(ObjectId oid, _syncSlave.values())
-    slaveCount += oid.count();
+    foreach (ObjectId oid, _syncSlave.values())
+        slaveCount += oid.count();
 
     qDebug() << this;
     qDebug() << "              Proxy Mode:" << mode;
@@ -814,13 +791,12 @@ void SignalProxy::dumpProxyStats()
     qDebug() << "number of Classes cached:" << _extendedMetaObjects.count();
 }
 
-
 void SignalProxy::updateSecureState()
 {
     bool wasSecure = _secure;
 
     _secure = !_peerMap.isEmpty();
-    for (auto peer :  _peerMap.values()) {
+    for (auto peer : _peerMap.values()) {
         _secure &= peer->isSecure();
     }
 
@@ -828,9 +804,10 @@ void SignalProxy::updateSecureState()
         emit secureStateChanged(_secure);
 }
 
-QVariantList SignalProxy::peerData() {
+QVariantList SignalProxy::peerData()
+{
     QVariantList result;
-    for (auto &&peer : _peerMap.values()) {
+    for (auto&& peer : _peerMap.values()) {
         QVariantMap data;
         data["id"] = peer->id();
         data["clientVersion"] = peer->clientVersion();
@@ -847,7 +824,8 @@ QVariantList SignalProxy::peerData() {
     return result;
 }
 
-Peer *SignalProxy::peerById(int peerId) {
+Peer* SignalProxy::peerById(int peerId)
+{
     // We use ::value() here instead of the [] operator because the latter has the side-effect
     // of automatically inserting a null value with the passed key into the map.  See
     // https://doc.qt.io/qt-5/qhash.html#operator-5b-5d and https://doc.qt.io/qt-5/qhash.html#value.
@@ -867,28 +845,32 @@ void SignalProxy::restrictTargetPeers(QSet<Peer*> peers, std::function<void()> c
     _restrictedTargets = previousRestrictedTargets;
 }
 
-Peer *SignalProxy::sourcePeer() {
+Peer* SignalProxy::sourcePeer()
+{
     return _sourcePeer;
 }
 
-void SignalProxy::setSourcePeer(Peer *sourcePeer) {
+void SignalProxy::setSourcePeer(Peer* sourcePeer)
+{
     _sourcePeer = sourcePeer;
 }
 
-Peer *SignalProxy::targetPeer() {
+Peer* SignalProxy::targetPeer()
+{
     return _targetPeer;
 }
 
-void SignalProxy::setTargetPeer(Peer *targetPeer) {
+void SignalProxy::setTargetPeer(Peer* targetPeer)
+{
     _targetPeer = targetPeer;
 }
 
 // ==================================================
 //  ExtendedMetaObject
 // ==================================================
-SignalProxy::ExtendedMetaObject::ExtendedMetaObject(const QMetaObject *meta, bool checkConflicts)
-    : _meta(meta),
-    _updatedRemotelyId(_meta->indexOfSignal("updatedRemotely()"))
+SignalProxy::ExtendedMetaObject::ExtendedMetaObject(const QMetaObjectmeta, bool checkConflicts)
+    : _meta(meta)
+    _updatedRemotelyId(_meta->indexOfSignal("updatedRemotely()"))
 {
     for (int i = 0; i < _meta->methodCount(); i++) {
         if (_meta->method(i).methodType() != QMetaMethod::Slot)
@@ -905,8 +887,8 @@ SignalProxy::ExtendedMetaObject::ExtendedMetaObject(const QMetaObject *meta, boo
             /* funny... moc creates for methods containing default parameters multiple metaMethod with separate methodIds.
                we don't care... we just need the full fledged version
              */
-            const QMetaMethod &current = _meta->method(_methodIds[method]);
-            const QMetaMethod &candidate = _meta->method(i);
+            const QMetaMethodcurrent = _meta->method(_methodIds[method]);
+            const QMetaMethodcandidate = _meta->method(i);
             if (current.parameterTypes().count() > candidate.parameterTypes().count()) {
                 int minCount = candidate.parameterTypes().count();
                 QList<QByteArray> commonParams = current.parameterTypes().mid(0, minCount);
@@ -917,13 +899,14 @@ SignalProxy::ExtendedMetaObject::ExtendedMetaObject(const QMetaObject *meta, boo
                 int minCount = current.parameterTypes().count();
                 QList<QByteArray> commonParams = candidate.parameterTypes().mid(0, minCount);
                 if (commonParams == current.parameterTypes()) {
-                    _methodIds[method] = i; // use the new one
+                    _methodIds[method] = i;  // use the new one
                     continue;
                 }
             }
             if (checkConflicts) {
                 qWarning() << "class" << meta->className() << "contains overloaded methods which is currently not supported!";
-                qWarning() << " - " << _meta->method(i).methodSignature() << "conflicts with" << _meta->method(_methodIds[method]).methodSignature();
+                qWarning() << " - " << _meta->method(i).methodSignature() << "conflicts with"
+                           << _meta->method(_methodIds[method]).methodSignature();
             }
             continue;
         }
@@ -931,8 +914,7 @@ SignalProxy::ExtendedMetaObject::ExtendedMetaObject(const QMetaObject *meta, boo
     }
 }
 
-
-const SignalProxy::ExtendedMetaObject::MethodDescriptor &SignalProxy::ExtendedMetaObject::methodDescriptor(int methodId)
+const SignalProxy::ExtendedMetaObject::MethodDescriptor& SignalProxy::ExtendedMetaObject::methodDescriptor(int methodId)
 {
     if (!_methods.contains(methodId)) {
         _methods[methodId] = MethodDescriptor(_meta->method(methodId));
@@ -940,8 +922,7 @@ const SignalProxy::ExtendedMetaObject::MethodDescriptor &SignalProxy::ExtendedMe
     return _methods[methodId];
 }
 
-
-const QHash<int, int> &SignalProxy::ExtendedMetaObject::receiveMap()
+const QHash<int, int>& SignalProxy::ExtendedMetaObject::receiveMap()
 {
     if (_receiveMap.isEmpty()) {
         QHash<int, int> receiveMap;
@@ -994,15 +975,13 @@ const QHash<int, int> &SignalProxy::ExtendedMetaObject::receiveMap()
     return _receiveMap;
 }
 
-
-QByteArray SignalProxy::ExtendedMetaObject::methodName(const QMetaMethod &method)
+QByteArray SignalProxy::ExtendedMetaObject::methodName(const QMetaMethod& method)
 {
     QByteArray sig(method.methodSignature());
     return sig.left(sig.indexOf("("));
 }
 
-
-QString SignalProxy::ExtendedMetaObject::methodBaseName(const QMetaMethod &method)
+QString SignalProxy::ExtendedMetaObject::methodBaseName(const QMetaMethod& method)
 {
     QString methodname = QString(method.methodSignature()).section("(", 0, 0);
 
@@ -1028,10 +1007,9 @@ QString SignalProxy::ExtendedMetaObject::methodBaseName(const QMetaMethod &metho
     return methodname;
 }
 
-
-SignalProxy::ExtendedMetaObject::MethodDescriptor::MethodDescriptor(const QMetaMethod &method)
-    : _methodName(SignalProxy::ExtendedMetaObject::methodName(method)),
-    _returnType(QMetaType::type(method.typeName()))
+SignalProxy::ExtendedMetaObject::MethodDescriptor::MethodDescriptor(const QMetaMethod& method)
+    : _methodName(SignalProxy::ExtendedMetaObject::methodName(method))
+    , _returnType(QMetaType::type(method.typeName()))
 {
     // determine argTypes
     QList<QByteArray> paramTypes = method.parameterTypes();
@@ -1045,7 +1023,5 @@ SignalProxy::ExtendedMetaObject::MethodDescriptor::MethodDescriptor(const QMetaM
     QString signature(method.methodSignature());
     _minArgCount = method.parameterTypes().count() - signature.count("=");
 
-    _receiverMode = (_methodName.startsWith("request"))
-                    ? SignalProxy::Server
-                    : SignalProxy::Client;
+    _receiverMode = (_methodName.startsWith("request")) ? SignalProxy::Server : SignalProxy::Client;
 }