removed compat class for SignalVector

src/common/SignalVector.hpp

@@ -171,15 +171,4 @@ private:
     std::function<bool(const T &, const T &)> itemCompare_;
 };
 
-// compatability
-template <typename T, typename Compare>
-class SortedSignalVector : public SignalVector<T>
-{
-public:
-    SortedSignalVector()
-        : SignalVector<T>(Compare{})
-    {
-    }
-};
-
 }  // namespace chatterino

src/controllers/accounts/AccountController.cpp

@@ -7,6 +7,7 @@
 namespace chatterino {
 
 AccountController::AccountController()
+    : accounts_(SharedPtrElementLess<Account>{})
 {
     this->twitch.accounts.itemInserted.connect([this](const auto &args) {
         this->accounts_.insert(std::dynamic_pointer_cast<Account>(args.item));

src/controllers/accounts/AccountController.hpp

@@ -27,8 +27,7 @@ public:
     TwitchAccountManager twitch;
 
 private:
-    SortedSignalVector<std::shared_ptr<Account>, SharedPtrElementLess<Account>>
-        accounts_;
+    SignalVector<std::shared_ptr<Account>> accounts_;
 };
 
 }  // namespace chatterino

src/controllers/taggedusers/TaggedUsersController.cpp

@@ -5,6 +5,7 @@
 namespace chatterino {
 
 TaggedUsersController::TaggedUsersController()
+    : users(std::less<TaggedUser>{})
 {
 }
 

src/controllers/taggedusers/TaggedUsersController.hpp

@@ -14,7 +14,7 @@ class TaggedUsersController final : public Singleton
 public:
     TaggedUsersController();
 
-    SortedSignalVector<TaggedUser, std::less<TaggedUser>> users;
+    SignalVector<TaggedUser> users;
 
     TaggedUsersModel *createModel(QObject *parent = nullptr);
 };

src/providers/twitch/TwitchAccountManager.cpp

@@ -7,7 +7,8 @@
 namespace chatterino {
 
 TwitchAccountManager::TwitchAccountManager()
-    : anonymousUser_(new TwitchAccount(ANONYMOUS_USERNAME, "", "", ""))
+    : accounts(SharedPtrElementLess<TwitchAccount>{})
+    , anonymousUser_(new TwitchAccount(ANONYMOUS_USERNAME, "", "", ""))
 {
     this->currentUserChanged.connect([this] {
         auto currentUser = this->getCurrent();

src/providers/twitch/TwitchAccountManager.hpp

@@ -51,9 +51,7 @@ public:
     pajlada::Signals::NoArgSignal currentUserChanged;
     pajlada::Signals::NoArgSignal userListUpdated;
 
-    SortedSignalVector<std::shared_ptr<TwitchAccount>,
-                       SharedPtrElementLess<TwitchAccount>>
-        accounts;
+    SignalVector<std::shared_ptr<TwitchAccount>> accounts;
 
 private:
     enum class AddUserResponse {