Make SettingsManager::setDownloadDir a void function

The current implementation doesn't have any effect on it being a bool
This commit is contained in:
Nikhil Tanwar 2022-02-08 20:09:53 +05:30 committed by Emmanuel Engelhart
parent a887af01be
commit 8d6d6833e7
No known key found for this signature in database
GPG Key ID: 120B30D020B553D3
2 changed files with 2 additions and 3 deletions

View File

@ -73,12 +73,11 @@ void SettingsManager::setZoomFactor(qreal zoomFactor)
emit(zoomChanged(zoomFactor)); emit(zoomChanged(zoomFactor));
} }
bool SettingsManager::setDownloadDir(QString downloadDir) void SettingsManager::setDownloadDir(QString downloadDir)
{ {
m_downloadDir = downloadDir; m_downloadDir = downloadDir;
m_settings.setValue("download/dir", downloadDir); m_settings.setValue("download/dir", downloadDir);
emit(downloadDirChanged(downloadDir)); emit(downloadDirChanged(downloadDir));
return true;
} }
void SettingsManager::initSettings() void SettingsManager::initSettings()

View File

@ -31,7 +31,7 @@ public slots:
void setKiwixServerPort(int port); void setKiwixServerPort(int port);
void setKiwixServerIpAddress(QString ipAddress); void setKiwixServerIpAddress(QString ipAddress);
void setZoomFactor(qreal zoomFactor); void setZoomFactor(qreal zoomFactor);
bool setDownloadDir(QString downloadDir); void setDownloadDir(QString downloadDir);
private: private:
void initSettings(); void initSettings();