Don't save connection state when destructing the controller

This causes neochat to crash and is done automatically by the connections

BUG: 458353

src/controller.cpp

@@ -125,13 +125,6 @@ Controller::Controller(QObject *parent)
 #endif
 }
 
-Controller::~Controller()
-{
-    for (auto c : AccountRegistry::instance().accounts()) {
-        c->saveState();
-    }
-}
-
 Controller &Controller::instance()
 {
     static Controller _instance;

src/controller.h

@@ -101,7 +101,6 @@ public:
 
 private:
     explicit Controller(QObject *parent = nullptr);
-    ~Controller() override;
 
     QPointer<Connection> m_connection;
     bool m_busy = false;