make lint

include/ChatPage.h

@@ -22,6 +22,7 @@
 #include <QWidget>
 
 #include "Cache.h"
+#include "CommunitiesList.h"
 #include "MatrixClient.h"
 #include "QuickSwitcher.h"
 #include "RoomList.h"
@@ -33,7 +34,6 @@
 #include "TopRoomBar.h"
 #include "TypingDisplay.h"
 #include "UserInfoWidget.h"
-#include "CommunitiesList.h"
 
 constexpr int CONSENSUS_TIMEOUT    = 1000;
 constexpr int SHOW_CONTENT_TIMEOUT = 3000;

include/CommunitiesList.h

@@ -1,13 +1,13 @@
 #pragma once
 
-#include <QWidget>
 #include <QScrollArea>
 #include <QSharedPointer>
 #include <QVBoxLayout>
+#include <QWidget>
 
-#include "MatrixClient.h"
 #include "CommunitiesListItem.h"
 #include "Community.h"
+#include "MatrixClient.h"
 #include "ui/Theme.h"
 
 class CommunitiesList : public QWidget
@@ -21,15 +21,13 @@ public:
         void setCommunities(const QMap<QString, QSharedPointer<Community>> &communities);
         void clear();
 
-    void addCommunity(QSharedPointer<Community> community,
+        void addCommunity(QSharedPointer<Community> community, const QString &community_id);
-                      const QString &community_id);
         void removeCommunity(const QString &community_id);
 signals:
         void communityChanged(const QString &community_id);
 
 public slots:
-    void updateCommunityAvatar(const QString &community_id,
+        void updateCommunityAvatar(const QString &community_id, const QPixmap &img);
-                               const QPixmap &img);
         void highlightSelectedCommunity(const QString &community_id);
 
 private:

include/CommunitiesListItem.h

@@ -1,14 +1,14 @@
 #pragma once
 
-#include <QWidget>
+#include <QDebug>
-#include <QSharedPointer>
 #include <QMouseEvent>
 #include <QPainter>
-#include <QDebug>
+#include <QSharedPointer>
+#include <QWidget>
 
-#include "ui/Theme.h"
-#include "Menu.h"
 #include "Community.h"
+#include "Menu.h"
+#include "ui/Theme.h"
 
 class CommunitiesListItem : public QWidget
 {
@@ -61,10 +61,7 @@ inline void
 CommunitiesListItem::setAvatar(const QImage &avatar_image)
 {
         communityAvatar_ = QPixmap::fromImage(
-                avatar_image.scaled(IconSize,
+          avatar_image.scaled(IconSize, IconSize, Qt::IgnoreAspectRatio, Qt::SmoothTransformation));
-                                    IconSize,
-                                    Qt::IgnoreAspectRatio,
-                                    Qt::SmoothTransformation));
         update();
 }
 
@@ -78,6 +75,7 @@ public:
 protected:
         void mousePressEvent(QMouseEvent *event) override;
         void paintEvent(QPaintEvent *event) override;
+
 private:
         const int IconSize = 55;
 };

include/Community.h

@@ -1,9 +1,9 @@
 #pragma once
 
-#include <QObject>
 #include <QJsonObject>
-#include <QUrl>
+#include <QObject>
 #include <QString>
+#include <QUrl>
 
 class Community : public QObject
 {
@@ -55,7 +55,6 @@ Community::getLongDescription() const
         return long_description_;
 }
 
-
 inline const QList<QString>
 Community::getRoomList() const
 {

include/Deserializable.h

@@ -37,15 +37,25 @@ private:
 class Deserializable
 {
 public:
-        virtual void deserialize(const QJsonValue &) {}
+        virtual void deserialize(const QJsonValue &)
-        virtual void deserialize(const QJsonObject &) {}
+        {
-        virtual void deserialize(const QJsonDocument &) {}
+        }
-        virtual ~Deserializable() {}
+        virtual void deserialize(const QJsonObject &)
+        {
+        }
+        virtual void deserialize(const QJsonDocument &)
+        {
+        }
+        virtual ~Deserializable()
+        {
+        }
 };
 
 class Serializable
 {
 public:
         virtual QJsonObject serialize() const = 0;
-        virtual ~Serializable() {}
+        virtual ~Serializable()
+        {
+        }
 };

include/RoomList.h

@@ -85,7 +85,7 @@ private:
         QString selectedRoom_;
 
         bool filterRooms_          = false;
-        QList<QString> roomFilter_ = QList<QString>(); //which rooms to include in the room list
+        QList<QString> roomFilter_ = QList<QString>(); // which rooms to include in the room list
 
         QSharedPointer<MatrixClient> client_;
 };

include/events/AliasesEventContent.h

@@ -26,9 +26,7 @@ namespace matrix
 {
 namespace events
 {
-class AliasesEventContent
+class AliasesEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/AvatarEventContent.h

@@ -30,9 +30,7 @@ namespace events
  * A picture that is associated with the room.
  */
 
-class AvatarEventContent
+class AvatarEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/CanonicalAliasEventContent.h

@@ -32,9 +32,7 @@ namespace events
  * users which alias to use to advertise the room.
  */
 
-class CanonicalAliasEventContent
+class CanonicalAliasEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/CreateEventContent.h

@@ -30,9 +30,7 @@ namespace events
  * of all other events.
  */
 
-class CreateEventContent
+class CreateEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/Event.h

@@ -62,9 +62,7 @@ bool
 isStateEvent(EventType type);
 
 template<class Content>
-class Event
+class Event : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         inline Content content() const;

include/events/HistoryVisibilityEventContent.h

@@ -32,9 +32,7 @@ enum class HistoryVisibility {
         WorldReadable,
 };
 
-class HistoryVisibilityEventContent
+class HistoryVisibilityEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         inline HistoryVisibility historyVisibility() const;

include/events/JoinRulesEventContent.h

@@ -44,9 +44,7 @@ enum class JoinRule {
  * Describes how users are allowed to join the room.
  */
 
-class JoinRulesEventContent
+class JoinRulesEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/MemberEventContent.h

@@ -47,9 +47,7 @@ enum class Membership {
  * The current membership state of a user in the room.
  */
 
-class MemberEventContent
+class MemberEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/MessageEventContent.h

@@ -57,9 +57,7 @@ enum class MessageEventType {
 MessageEventType
 extractMessageEventType(const QJsonObject &data);
 
-class MessageEventContent
+class MessageEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/NameEventContent.h

@@ -29,9 +29,7 @@ namespace events
  * A human-friendly room name designed to be displayed to the end-user.
  */
 
-class NameEventContent
+class NameEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/PowerLevelsEventContent.h

@@ -36,9 +36,7 @@ enum class PowerLevels {
  * Defines the power levels (privileges) of users in the room.
  */
 
-class PowerLevelsEventContent
+class PowerLevelsEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

include/events/TopicEventContent.h

@@ -30,9 +30,7 @@ namespace events
  * room.
  */
 
-class TopicEventContent
+class TopicEventContent : public Deserializable, public Serializable
-  : public Deserializable
-  , public Serializable
 {
 public:
         void deserialize(const QJsonValue &data) override;

src/ChatPage.cc

@@ -49,7 +49,7 @@ ChatPage::ChatPage(QSharedPointer<MatrixClient> client, QWidget *parent)
 
         communitiesList_ = new CommunitiesList(client, this);
         communitiesSideBarLayout_->addWidget(communitiesList_);
-        //communitiesSideBarLayout_->addStretch(1);
+        // communitiesSideBarLayout_->addStretch(1);
         topLayout_->addWidget(communitiesSideBar_);
 
         auto splitter = new Splitter(this);
@@ -220,11 +220,15 @@ ChatPage::ChatPage(QSharedPointer<MatrixClient> client, QWidget *parent)
                 SIGNAL(getOwnCommunitiesResponse(QList<QString>)),
                 this,
                 SLOT(updateOwnCommunitiesInfo(QList<QString>)));
-        connect(client_.data(), &MatrixClient::communityProfileRetrieved, this,
+        connect(client_.data(),
+                &MatrixClient::communityProfileRetrieved,
+                this,
                 [=](QString communityId, QJsonObject profile) {
                         communityManager_[communityId]->parseProfile(profile);
                 });
-        connect(client_.data(), &MatrixClient::communityRoomsRetrieved, this,
+        connect(client_.data(),
+                &MatrixClient::communityRoomsRetrieved,
+                this,
                 [=](QString communityId, QJsonObject rooms) {
                         communityManager_[communityId]->parseRooms(rooms);
 
@@ -232,7 +236,8 @@ ChatPage::ChatPage(QSharedPointer<MatrixClient> client, QWidget *parent)
                                 if (communityId == "world") {
                                         room_list_->setFilterRooms(false);
                                 } else {
-                    room_list_->setRoomFilter(communityManager_[communityId]->getRoomList());
+                                        room_list_->setRoomFilter(
+                                          communityManager_[communityId]->getRoomList());
                                 }
                         }
                 });
@@ -265,12 +270,16 @@ ChatPage::ChatPage(QSharedPointer<MatrixClient> client, QWidget *parent)
                 }
         });
 
-        connect(communitiesList_, &CommunitiesList::communityChanged, this, [=](const QString &communityId) {
+        connect(communitiesList_,
+                &CommunitiesList::communityChanged,
+                this,
+                [=](const QString &communityId) {
                         current_community_ = communityId;
                         if (communityId == "world") {
                                 room_list_->setFilterRooms(false);
                         } else {
-                room_list_->setRoomFilter(communityManager_[communityId]->getRoomList());
+                                room_list_->setRoomFilter(
+                                  communityManager_[communityId]->getRoomList());
                         }
                 });
 

src/CommunitiesList.cc

@@ -35,35 +35,44 @@ CommunitiesList::CommunitiesList(QSharedPointer<MatrixClient> client, QWidget *p
         WorldCommunityListItem *world_list_item = new WorldCommunityListItem();
         contentsLayout_->addWidget(world_list_item);
         communities_.insert("world", QSharedPointer<CommunitiesListItem>(world_list_item));
-    connect(world_list_item, &WorldCommunityListItem::clicked,
+        connect(world_list_item,
-            this, &CommunitiesList::highlightSelectedCommunity);
+                &WorldCommunityListItem::clicked,
+                this,
+                &CommunitiesList::highlightSelectedCommunity);
         contentsLayout_->addStretch(1);
 
         scrollArea_->setWidget(scrollAreaContents_);
         topLayout_->addWidget(scrollArea_);
 
-    connect(client_.data(), &MatrixClient::communityProfileRetrieved, this,
+        connect(client_.data(),
+                &MatrixClient::communityProfileRetrieved,
+                this,
                 [=](QString communityId, QJsonObject profile) {
-        client_->fetchCommunityAvatar(communityId, QUrl(profile["avatar_url"].toString()));
+                        client_->fetchCommunityAvatar(communityId,
+                                                      QUrl(profile["avatar_url"].toString()));
                 });
         connect(client_.data(),
                 SIGNAL(communityAvatarRetrieved(const QString &, const QPixmap &)),
                 this,
-            SLOT(updateCommunityAvatar(const QString &,const QPixmap &)));
+                SLOT(updateCommunityAvatar(const QString &, const QPixmap &)));
 }
 
-CommunitiesList::~CommunitiesList() {}
+CommunitiesList::~CommunitiesList()
+{
+}
 
 void
 CommunitiesList::setCommunities(const QMap<QString, QSharedPointer<Community>> &communities)
 {
         communities_.clear();
 
-    //TODO: still not sure how to handle the "world" special-case
+        // TODO: still not sure how to handle the "world" special-case
         WorldCommunityListItem *world_list_item = new WorldCommunityListItem();
         communities_.insert("world", QSharedPointer<CommunitiesListItem>(world_list_item));
-    connect(world_list_item, &WorldCommunityListItem::clicked,
+        connect(world_list_item,
-            this, &CommunitiesList::highlightSelectedCommunity);
+                &WorldCommunityListItem::clicked,
+                this,
+                &CommunitiesList::highlightSelectedCommunity);
         contentsLayout_->insertWidget(0, world_list_item);
 
         for (auto it = communities.constBegin(); it != communities.constEnd(); it++) {
@@ -89,18 +98,19 @@ CommunitiesList::clear()
 void
 CommunitiesList::addCommunity(QSharedPointer<Community> community, const QString &community_id)
 {
-    CommunitiesListItem *list_item = new CommunitiesListItem(community,
+        CommunitiesListItem *list_item =
-                                                             community_id,
+          new CommunitiesListItem(community, community_id, scrollArea_);
-                                                             scrollArea_);
 
         communities_.insert(community_id, QSharedPointer<CommunitiesListItem>(list_item));
 
         client_->fetchCommunityAvatar(community_id, community->getAvatar());
 
-    contentsLayout_->insertWidget(contentsLayout_->count()-1, list_item);
+        contentsLayout_->insertWidget(contentsLayout_->count() - 1, list_item);
 
-    connect(list_item, &CommunitiesListItem::clicked,
+        connect(list_item,
-            this, &CommunitiesList::highlightSelectedCommunity);
+                &CommunitiesListItem::clicked,
+                this,
+                &CommunitiesList::highlightSelectedCommunity);
 }
 
 void
@@ -118,7 +128,6 @@ CommunitiesList::updateCommunityAvatar(const QString &community_id, const QPixma
         }
 
         communities_.value(community_id)->setAvatar(img.toImage());
-
 }
 
 void

src/CommunitiesListItem.cc

@@ -7,13 +7,15 @@ CommunitiesListItem::CommunitiesListItem(QSharedPointer<Community> community,
   , community_(community)
   , communityId_(community_id)
 {
-    //menu_ = new Menu(this);
+        // menu_ = new Menu(this);
         setSizePolicy(QSizePolicy::Fixed, QSizePolicy::Fixed);
         setFixedHeight(ui::sidebar::CommunitiesSidebarSize);
         setFixedWidth(ui::sidebar::CommunitiesSidebarSize);
 }
 
-CommunitiesListItem::~CommunitiesListItem() {}
+CommunitiesListItem::~CommunitiesListItem()
+{
+}
 
 void
 CommunitiesListItem::setCommunity(QSharedPointer<Community> community)
@@ -31,7 +33,8 @@ CommunitiesListItem::setPressedState(bool state)
 }
 
 void
-CommunitiesListItem::mousePressEvent(QMouseEvent *event) {
+CommunitiesListItem::mousePressEvent(QMouseEvent *event)
+{
         if (event->buttons() == Qt::RightButton) {
                 QWidget::mousePressEvent(event);
                 return;
@@ -47,7 +50,6 @@ CommunitiesListItem::paintEvent(QPaintEvent *event)
 {
         Q_UNUSED(event);
 
-
         QPainter p(this);
         p.setRenderHint(QPainter::TextAntialiasing);
         p.setRenderHint(QPainter::SmoothPixmapTransform);
@@ -65,7 +67,7 @@ CommunitiesListItem::paintEvent(QPaintEvent *event)
 
         p.setPen(QColor("#333"));
 
-        QRect avatarRegion((width()-IconSize)/2, (height()-IconSize)/2, IconSize, IconSize);
+        QRect avatarRegion((width() - IconSize) / 2, (height() - IconSize) / 2, IconSize, IconSize);
 
         font.setBold(false);
         p.setPen(Qt::NoPen);
@@ -91,14 +93,15 @@ CommunitiesListItem::paintEvent(QPaintEvent *event)
                 p.save();
 
                 QPainterPath path;
-                path.addEllipse((width()-IconSize)/2, (height()-IconSize)/2, IconSize, IconSize);
+                path.addEllipse(
+                  (width() - IconSize) / 2, (height() - IconSize) / 2, IconSize, IconSize);
                 p.setClipPath(path);
 
                 p.drawPixmap(avatarRegion, communityAvatar_);
                 p.restore();
         }
 
-        //TODO: Discord-style community ping counts?
+        // TODO: Discord-style community ping counts?
         /*if (unreadMsgCount_ > 0) {
                 QColor textColor("white");
                 QColor backgroundColor("#38A3D8");
@@ -142,7 +145,7 @@ CommunitiesListItem::contextMenuEvent(QContextMenuEvent *event)
 {
         Q_UNUSED(event);
 
-        //menu_->popup(event->globalPos());
+        // menu_->popup(event->globalPos());
 }
 
 WorldCommunityListItem::WorldCommunityListItem(QWidget *parent)
@@ -150,7 +153,9 @@ WorldCommunityListItem::WorldCommunityListItem(QWidget *parent)
 {
 }
 
-WorldCommunityListItem::~WorldCommunityListItem() {}
+WorldCommunityListItem::~WorldCommunityListItem()
+{
+}
 
 void
 WorldCommunityListItem::mousePressEvent(QMouseEvent *event)
@@ -190,10 +195,10 @@ WorldCommunityListItem::paintEvent(QPaintEvent *event)
         p.setPen(Qt::NoPen);
         p.setBrush(brush);
 
-        QRect avatarRegion((width()-IconSize)/2, (height()-IconSize)/2, IconSize, IconSize);
+        QRect avatarRegion((width() - IconSize) / 2, (height() - IconSize) / 2, IconSize, IconSize);
         p.drawEllipse(avatarRegion.center(), IconSize / 2, IconSize / 2);
         QPainterPath path;
-        path.addEllipse((width()-IconSize)/2, (height()-IconSize)/2, IconSize, IconSize);
+        path.addEllipse((width() - IconSize) / 2, (height() - IconSize) / 2, IconSize, IconSize);
         p.setClipPath(path);
 
         p.drawPixmap(avatarRegion, worldIcon);

src/Community.cc

@@ -1,7 +1,7 @@
 #include "include/Community.h"
 
-#include <QJsonValue>
 #include <QJsonArray>
+#include <QJsonValue>
 
 void
 Community::parseProfile(const QJsonObject &profile)
@@ -9,7 +9,7 @@ Community::parseProfile(const QJsonObject &profile)
         if (profile["name"].type() == QJsonValue::Type::String) {
                 name_ = profile["name"].toString();
         } else {
-        name_ = "Unnamed Community"; //TODO: what is correct here?
+                name_ = "Unnamed Community"; // TODO: what is correct here?
         }
 
         if (profile["avatar_url"].type() == QJsonValue::Type::String) {
@@ -29,7 +29,6 @@ Community::parseProfile(const QJsonObject &profile)
         } else {
                 long_description_ = "";
         }
-
 }
 
 void
@@ -37,7 +36,7 @@ Community::parseRooms(const QJsonObject &rooms)
 {
         rooms_.clear();
 
-    for (auto i = 0; i<rooms["chunk"].toArray().size(); i++) {
+        for (auto i = 0; i < rooms["chunk"].toArray().size(); i++) {
                 rooms_.append(rooms["chunk"].toArray()[i].toObject()["room_id"].toString());
         }
 

src/EmojiCategory.cc

@@ -80,4 +80,6 @@ EmojiCategory::EmojiCategory(QString category, QList<Emoji> emoji, QWidget *pare
         connect(emojiListView_, &QListView::clicked, this, &EmojiCategory::clickIndex);
 }
 
-EmojiCategory::~EmojiCategory() {}
+EmojiCategory::~EmojiCategory()
+{
+}

src/EmojiProvider.cc

@@ -485,7 +485,8 @@ const QList<Emoji> EmojiProvider::food = {
         Emoji{ QString::fromUtf8("\xf0\x9f\xa5\x84"), ":spoon:" },
 };
 
-const QList<Emoji> EmojiProvider::activity = {
+const QList<Emoji> EmojiProvider::activity =
+  {
     Emoji{ QString::fromUtf8("\xf0\x9f\x91\xbe"), ":space_invader:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\x95\xb4"), ":levitate:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\xa4\xba"), ":fencer:" },
@@ -535,16 +536,11 @@ const QList<Emoji> EmojiProvider::activity = {
     Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb4\xf0\x9f\x8f\xbe"), ":bicyclist_tone4:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb4\xf0\x9f\x8f\xbf"), ":bicyclist_tone5:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5"), ":mountain_bicyclist:" },
-        Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbb"),
+    Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbb"), ":mountain_bicyclist_tone1:" },
-               ":mountain_bicyclist_tone1:" },
+    Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbc"), ":mountain_bicyclist_tone2:" },
-        Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbc"),
+    Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbd"), ":mountain_bicyclist_tone3:" },
-               ":mountain_bicyclist_tone2:" },
+    Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbe"), ":mountain_bicyclist_tone4:" },
-        Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbd"),
+    Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbf"), ":mountain_bicyclist_tone5:" },
-               ":mountain_bicyclist_tone3:" },
-        Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbe"),
-               ":mountain_bicyclist_tone4:" },
-        Emoji{ QString::fromUtf8("\xf0\x9f\x9a\xb5\xf0\x9f\x8f\xbf"),
-               ":mountain_bicyclist_tone5:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\xa4\xb8"), ":cartwheel:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\xa4\xb8\xf0\x9f\x8f\xbb"), ":cartwheel_tone1:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\xa4\xb8\xf0\x9f\x8f\xbc"), ":cartwheel_tone2:" },
@@ -630,7 +626,7 @@ const QList<Emoji> EmojiProvider::activity = {
     Emoji{ QString::fromUtf8("\xf0\x9f\xa5\x81"), ":drum:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\x8e\xac"), ":clapper:" },
     Emoji{ QString::fromUtf8("\xf0\x9f\x8f\xb9"), ":bow_and_arrow:" },
-};
+  };
 
 const QList<Emoji> EmojiProvider::travel = {
         Emoji{ QString::fromUtf8("\xf0\x9f\x8f\x8e"), ":race_car:" },

src/Login.cc

@@ -22,7 +22,9 @@
 #include "Deserializable.h"
 #include "Login.h"
 
-LoginRequest::LoginRequest() {}
+LoginRequest::LoginRequest()
+{
+}
 
 LoginRequest::LoginRequest(QString username, QString password)
   : user_(username)

src/LoginPage.cc

@@ -297,4 +297,6 @@ LoginPage::onBackButtonClicked()
         emit backButtonClicked();
 }
 
-LoginPage::~LoginPage() {}
+LoginPage::~LoginPage()
+{
+}

src/MainWindow.cc

@@ -253,4 +253,6 @@ MainWindow::instance()
         return instance_;
 }
 
-MainWindow::~MainWindow() {}
+MainWindow::~MainWindow()
+{
+}

src/MatrixClient.cc

@@ -230,7 +230,9 @@ MatrixClient::onGetOwnCommunitiesResponse(QNetworkReply *reply)
 
         try {
                 QList<QString> response;
-                for (auto it = json["groups"].toArray().constBegin(); it != json["groups"].toArray().constEnd(); it++) {
+                for (auto it = json["groups"].toArray().constBegin();
+                     it != json["groups"].toArray().constEnd();
+                     it++) {
                         response.append(it->toString());
                 }
                 emit getOwnCommunitiesResponse(response);

src/RegisterPage.cc

@@ -181,4 +181,6 @@ RegisterPage::onRegisterButtonClicked()
         }
 }
 
-RegisterPage::~RegisterPage() {}
+RegisterPage::~RegisterPage()
+{
+}

src/RoomInfoListItem.cc

@@ -310,4 +310,6 @@ RoomInfoListItem::mousePressEvent(QMouseEvent *event)
         ripple_overlay_->addRipple(ripple);
 }
 
-RoomInfoListItem::~RoomInfoListItem() {}
+RoomInfoListItem::~RoomInfoListItem()
+{
+}

src/RoomList.cc

@@ -62,7 +62,9 @@ RoomList::RoomList(QSharedPointer<MatrixClient> client, QWidget *parent)
                 SLOT(updateRoomAvatar(const QString &, const QPixmap &)));
 }
 
-RoomList::~RoomList() {}
+RoomList::~RoomList()
+{
+}
 
 void
 RoomList::clear()
@@ -291,11 +293,11 @@ RoomList::setFilterRooms(bool filterRooms)
 {
         filterRooms_ = filterRooms;
 
-        for (int i=0; i<contentsLayout_->count(); i++) {
+        for (int i = 0; i < contentsLayout_->count(); i++) {
-
+                // If roomFilter_ contains the room for the current RoomInfoListItem,
-            //If roomFilter_ contains the room for the current RoomInfoListItem,
+                // show the list item, otherwise hide it
-            //show the list item, otherwise hide it
+                RoomInfoListItem *listitem =
-            RoomInfoListItem *listitem = (RoomInfoListItem *) contentsLayout_->itemAt(i)->widget();
+                  (RoomInfoListItem *)contentsLayout_->itemAt(i)->widget();
 
                 if (listitem != nullptr) {
                         if (!filterRooms) {
@@ -310,10 +312,10 @@ RoomList::setFilterRooms(bool filterRooms)
 
         if (!roomFilter_.contains(selectedRoom_)) {
                 RoomInfoListItem *firstVisibleRoom = nullptr;
-            for (int i=0; i<contentsLayout_->count(); i++) {
+                for (int i = 0; i < contentsLayout_->count(); i++) {
                         QWidget *item = contentsLayout_->itemAt(i)->widget();
                         if (item != nullptr && item->isVisible()) {
-                    firstVisibleRoom = (RoomInfoListItem *) item;
+                                firstVisibleRoom = (RoomInfoListItem *)item;
                                 break;
                         }
                 }

src/TextInputWidget.cc

@@ -231,4 +231,6 @@ TextInputWidget::hideUploadSpinner()
         spinner_->stop();
 }
 
-TextInputWidget::~TextInputWidget() {}
+TextInputWidget::~TextInputWidget()
+{
+}

src/TimelineItem.cc

@@ -461,4 +461,6 @@ TimelineItem::descriptiveTime(const QDateTime &then)
         return then.toString("dd/MM/yy");
 }
 
-TimelineItem::~TimelineItem() {}
+TimelineItem::~TimelineItem()
+{
+}

src/TimelineViewManager.cc

@@ -37,7 +37,9 @@ TimelineViewManager::TimelineViewManager(QSharedPointer<MatrixClient> client, QW
           client_.data(), &MatrixClient::messageSent, this, &TimelineViewManager::messageSent);
 }
 
-TimelineViewManager::~TimelineViewManager() {}
+TimelineViewManager::~TimelineViewManager()
+{
+}
 
 void
 TimelineViewManager::messageSent(const QString &event_id, const QString &roomid, int txn_id)

src/TopRoomBar.cc

@@ -171,4 +171,6 @@ TopRoomBar::setRoomSettings(QSharedPointer<RoomSettings> settings)
         roomSettings_ = settings;
 }
 
-TopRoomBar::~TopRoomBar() {}
+TopRoomBar::~TopRoomBar()
+{
+}

src/UserInfoWidget.cc

@@ -121,7 +121,9 @@ UserInfoWidget::closeLogoutDialog(bool isLoggingOut)
                 emit logout();
 }
 
-UserInfoWidget::~UserInfoWidget() {}
+UserInfoWidget::~UserInfoWidget()
+{
+}
 
 void
 UserInfoWidget::resizeEvent(QResizeEvent *event)

src/ui/Avatar.cc

@@ -19,7 +19,9 @@ Avatar::Avatar(QWidget *parent)
         setSizePolicy(policy);
 }
 
-Avatar::~Avatar() {}
+Avatar::~Avatar()
+{
+}
 
 QColor
 Avatar::textColor() const

src/ui/Badge.cc

@@ -22,7 +22,9 @@ Badge::Badge(const QString &text, QWidget *parent)
         setText(text);
 }
 
-Badge::~Badge() {}
+Badge::~Badge()
+{
+}
 
 void
 Badge::init()

src/ui/FlatButton.cc

@@ -80,7 +80,9 @@ FlatButton::FlatButton(const QString &text, ui::Role role, QWidget *parent, ui::
         setRole(role);
 }
 
-FlatButton::~FlatButton() {}
+FlatButton::~FlatButton()
+{
+}
 
 void
 FlatButton::applyPreset(ui::ButtonPreset preset)
@@ -610,7 +612,9 @@ FlatButtonStateMachine::FlatButtonStateMachine(FlatButton *parent)
         addTransition(button_, QEvent::FocusOut, pressed_state_, hovered_state_);
 }
 
-FlatButtonStateMachine::~FlatButtonStateMachine() {}
+FlatButtonStateMachine::~FlatButtonStateMachine()
+{
+}
 
 void
 FlatButtonStateMachine::setOverlayOpacity(qreal opacity)

src/ui/RaisedButton.cc

@@ -70,7 +70,9 @@ RaisedButton::RaisedButton(const QString &text, QWidget *parent)
         setText(text);
 }
 
-RaisedButton::~RaisedButton() {}
+RaisedButton::~RaisedButton()
+{
+}
 
 bool
 RaisedButton::event(QEvent *event)

src/ui/Ripple.cc

@@ -25,7 +25,9 @@ Ripple::Ripple(const QPoint &center, RippleOverlay *overlay, QObject *parent)
         init();
 }
 
-Ripple::~Ripple() {}
+Ripple::~Ripple()
+{
+}
 
 void
 Ripple::setRadius(qreal radius)

src/ui/RippleOverlay.cc

@@ -11,7 +11,9 @@ RippleOverlay::RippleOverlay(QWidget *parent)
         setAttribute(Qt::WA_NoSystemBackground);
 }
 
-RippleOverlay::~RippleOverlay() {}
+RippleOverlay::~RippleOverlay()
+{
+}
 
 void
 RippleOverlay::addRipple(Ripple *ripple)

src/ui/TextField.cc

@@ -32,7 +32,9 @@ TextField::TextField(QWidget *parent)
         QCoreApplication::processEvents();
 }
 
-TextField::~TextField() {}
+TextField::~TextField()
+{
+}
 
 void
 TextField::setBackgroundColor(const QColor &color)
@@ -272,7 +274,9 @@ TextFieldStateMachine::TextFieldStateMachine(TextField *parent)
         connect(text_field_, SIGNAL(textChanged(QString)), this, SLOT(setupProperties()));
 }
 
-TextFieldStateMachine::~TextFieldStateMachine() {}
+TextFieldStateMachine::~TextFieldStateMachine()
+{
+}
 
 void
 TextFieldStateMachine::setLabel(TextFieldLabel *label)
@@ -349,7 +353,9 @@ TextFieldLabel::TextFieldLabel(TextField *parent)
         setFont(font);
 }
 
-TextFieldLabel::~TextFieldLabel() {}
+TextFieldLabel::~TextFieldLabel()
+{
+}
 
 void
 TextFieldLabel::paintEvent(QPaintEvent *)

src/ui/Theme.cc

@@ -22,7 +22,9 @@ Theme::Theme(QObject *parent)
         setColor("Transparent", ui::Color::Transparent);
 }
 
-Theme::~Theme() {}
+Theme::~Theme()
+{
+}
 
 QColor
 Theme::rgba(int r, int g, int b, qreal a) const