Browse Source

Fix singleton class ownership

We shouldn't allow Qt parent ownership in here.
adaptive-webui-19844
Chocobo1 5 years ago
parent
commit
a6cdba17f0
No known key found for this signature in database
GPG Key ID: 210D9C873253A68C
  1. 2
      src/app/application.cpp
  2. 10
      src/base/scanfoldersmodel.cpp
  3. 2
      src/base/scanfoldersmodel.h

2
src/app/application.cpp

@ -550,7 +550,7 @@ int Application::exec(const QStringList &params)
#ifndef DISABLE_COUNTRIES_RESOLUTION #ifndef DISABLE_COUNTRIES_RESOLUTION
Net::GeoIPManager::initInstance(); Net::GeoIPManager::initInstance();
#endif #endif
ScanFoldersModel::initInstance(this); ScanFoldersModel::initInstance();
#ifndef DISABLE_WEBUI #ifndef DISABLE_WEBUI
m_webui = new WebUI; m_webui = new WebUI;

10
src/base/scanfoldersmodel.cpp

@ -57,14 +57,10 @@ struct ScanFoldersModel::PathData
ScanFoldersModel *ScanFoldersModel::m_instance = nullptr; ScanFoldersModel *ScanFoldersModel::m_instance = nullptr;
bool ScanFoldersModel::initInstance(QObject *parent) void ScanFoldersModel::initInstance()
{ {
if (!m_instance) { if (!m_instance)
m_instance = new ScanFoldersModel(parent); m_instance = new ScanFoldersModel;
return true;
}
return false;
} }
void ScanFoldersModel::freeInstance() void ScanFoldersModel::freeInstance()

2
src/base/scanfoldersmodel.h

@ -65,7 +65,7 @@ public:
CUSTOM_LOCATION CUSTOM_LOCATION
}; };
static bool initInstance(QObject *parent = nullptr); static void initInstance();
static void freeInstance(); static void freeInstance();
static ScanFoldersModel *instance(); static ScanFoldersModel *instance();

Loading…
Cancel
Save