Clean up unused avatar functions

src/TopRoomBar.cpp

@@ -196,13 +196,6 @@ TopRoomBar::updateRoomAvatar(const QString &avatar_image)
         update();
 }
 
-void
-TopRoomBar::updateRoomAvatar(const QIcon &icon)
-{
-        avatar_->setIcon(icon);
-        update();
-}
-
 void
 TopRoomBar::updateRoomName(const QString &name)
 {

src/TopRoomBar.h

@@ -29,7 +29,6 @@ class OverlayModal;
 
 class QPainter;
 class QLabel;
-class QIcon;
 class QHBoxLayout;
 class QVBoxLayout;
 
@@ -43,7 +42,6 @@ public:
         TopRoomBar(QWidget *parent = nullptr);
 
         void updateRoomAvatar(const QString &avatar_image);
-        void updateRoomAvatar(const QIcon &icon);
         void updateRoomName(const QString &name);
         void updateRoomTopic(QString topic);
         void updateRoomAvatarFromName(const QString &name);

src/ui/Avatar.cpp

@@ -72,6 +72,8 @@ Avatar::setImage(const QString &avatar_url)
                                 static_cast<int>(size_ * pixmap_.devicePixelRatio()),
                                 this,
                                 [this](QPixmap pm) {
+                                        if (pm.isNull())
+                                                return;
                                         type_   = ui::AvatarType::Image;
                                         pixmap_ = pm;
                                         update();
@@ -88,20 +90,14 @@ Avatar::setImage(const QString &room, const QString &user)
                                 static_cast<int>(size_ * pixmap_.devicePixelRatio()),
                                 this,
                                 [this](QPixmap pm) {
+                                        if (pm.isNull())
+                                                return;
                                         type_   = ui::AvatarType::Image;
                                         pixmap_ = pm;
                                         update();
                                 });
 }
 
-void
-Avatar::setIcon(const QIcon &icon)
-{
-        icon_ = icon;
-        type_ = ui::AvatarType::Icon;
-        update();
-}
-
 void
 Avatar::paintEvent(QPaintEvent *)
 {
@@ -135,13 +131,6 @@ Avatar::paintEvent(QPaintEvent *)
         }
 
         switch (type_) {
-        case ui::AvatarType::Icon: {
-                icon_.paint(&painter,
-                            QRect((width() - hs) / 2, (height() - hs) / 2, hs, hs),
-                            Qt::AlignCenter,
-                            QIcon::Normal);
-                break;
-        }
         case ui::AvatarType::Image: {
                 QPainterPath ppath;
 

src/ui/Avatar.h

 #pragma once
 
-#include <QIcon>
 #include <QImage>
 #include <QPixmap>
 #include <QWidget>
@@ -18,7 +17,6 @@ public:
         explicit Avatar(QWidget *parent = nullptr, int size = ui::AvatarSize);
 
         void setBackgroundColor(const QColor &color);
-        void setIcon(const QIcon &icon);
         void setImage(const QString &avatar_url);
         void setImage(const QString &room, const QString &user);
         void setLetter(const QString &letter);
@@ -40,7 +38,6 @@ private:
         QString avatar_url_, room_, user_;
         QColor background_color_;
         QColor text_color_;
-        QIcon icon_;
         QPixmap pixmap_;
         int size_;
 };

src/ui/Theme.h

@@ -7,7 +7,6 @@
 namespace ui {
 enum class AvatarType
 {
-        Icon,
         Image,
         Letter
 };