pass settingsPath by const ref

cockatrice/src/settings/card_database_settings.cpp

@@ -1,6 +1,6 @@
 #include "card_database_settings.h"
 
-CardDatabaseSettings::CardDatabaseSettings(QString settingPath, QObject *parent)
+CardDatabaseSettings::CardDatabaseSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "cardDatabase.ini", parent)
 {
 }

cockatrice/src/settings/card_database_settings.h

@@ -25,7 +25,7 @@ signals:
 public slots:
 
 private:
-    explicit CardDatabaseSettings(QString settingPath, QObject *parent = nullptr);
+    explicit CardDatabaseSettings(const QString &settingPath, QObject *parent = nullptr);
     CardDatabaseSettings(const CardDatabaseSettings & /*other*/);
 };
 

cockatrice/src/settings/card_override_settings.cpp

@@ -1,6 +1,6 @@
 #include "card_override_settings.h"
 
-CardOverrideSettings::CardOverrideSettings(QString settingPath, QObject *parent)
+CardOverrideSettings::CardOverrideSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "cardPreferenceOverrides.ini", parent)
 {
 }

cockatrice/src/settings/card_override_settings.h

@@ -18,7 +18,7 @@ public:
     QString getCardPreferenceOverride(const QString &cardName);
 
 private:
-    explicit CardOverrideSettings(QString settingPath, QObject *parent = nullptr);
+    explicit CardOverrideSettings(const QString &settingPath, QObject *parent = nullptr);
     CardOverrideSettings(const CardOverrideSettings & /*other*/);
 };
 

cockatrice/src/settings/game_filters_settings.cpp

@@ -3,7 +3,7 @@
 #include <QCryptographicHash>
 #include <QTime>
 
-GameFiltersSettings::GameFiltersSettings(QString settingPath, QObject *parent)
+GameFiltersSettings::GameFiltersSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "gamefilters.ini", parent)
 {
 }

cockatrice/src/settings/game_filters_settings.h

@@ -46,7 +46,7 @@ signals:
 public slots:
 
 private:
-    explicit GameFiltersSettings(QString settingPath, QObject *parent = nullptr);
+    explicit GameFiltersSettings(const QString &settingPath, QObject *parent = nullptr);
     GameFiltersSettings(const GameFiltersSettings & /*other*/);
 
     QString hashGameType(const QString &gameType) const;

cockatrice/src/settings/message_settings.cpp

@@ -1,6 +1,6 @@
 #include "message_settings.h"
 
-MessageSettings::MessageSettings(QString settingPath, QObject *parent)
+MessageSettings::MessageSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "messages.ini", parent)
 {
 }

cockatrice/src/settings/message_settings.h

@@ -20,7 +20,7 @@ signals:
 public slots:
 
 private:
-    explicit MessageSettings(QString settingPath, QObject *parent = nullptr);
+    explicit MessageSettings(const QString &settingPath, QObject *parent = nullptr);
     MessageSettings(const MessageSettings & /*other*/);
 };
 

cockatrice/src/settings/recents_settings.cpp

@@ -2,7 +2,7 @@
 
 #define MAX_RECENT_DECK_COUNT 10
 
-RecentsSettings::RecentsSettings(QString settingPath, QObject *parent)
+RecentsSettings::RecentsSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "recents.ini", parent)
 {
 }

cockatrice/src/settings/recents_settings.h

@@ -8,7 +8,7 @@ class RecentsSettings : public SettingsManager
     Q_OBJECT
     friend class SettingsCache;
 
-    explicit RecentsSettings(QString settingPath, QObject *parent = nullptr);
+    explicit RecentsSettings(const QString &settingPath, QObject *parent = nullptr);
     RecentsSettings(const RecentsSettings & /*other*/);
 
 public:

cockatrice/src/settings/servers_settings.cpp

@@ -3,7 +3,7 @@
 #include <QDebug>
 #include <utility>
 
-ServersSettings::ServersSettings(QString settingPath, QObject *parent)
+ServersSettings::ServersSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "servers.ini", parent)
 {
 }

cockatrice/src/settings/servers_settings.h

@@ -61,7 +61,7 @@ public:
     bool getClearDebugLogStatus(bool abDefaultValue);
 
 private:
-    explicit ServersSettings(QString settingPath, QObject *parent = nullptr);
+    explicit ServersSettings(const QString &settingPath, QObject *parent = nullptr);
     ServersSettings(const ServersSettings & /*other*/);
 };
 

cockatrice/src/settings/settings_manager.cpp

@@ -1,6 +1,6 @@
 #include "settings_manager.h"
 
-SettingsManager::SettingsManager(QString settingPath, QObject *parent)
+SettingsManager::SettingsManager(const QString &settingPath, QObject *parent)
     : QObject(parent), settings(settingPath, QSettings::IniFormat)
 {
 }

cockatrice/src/settings/settings_manager.h

@@ -10,7 +10,7 @@ class SettingsManager : public QObject
 {
     Q_OBJECT
 public:
-    explicit SettingsManager(QString settingPath, QObject *parent = nullptr);
+    explicit SettingsManager(const QString &settingPath, QObject *parent = nullptr);
     QVariant getValue(QString name, QString group = "", QString subGroup = "");
     void sync();
 

dbconverter/src/mocks.cpp

@@ -1,7 +1,7 @@
 
 #include "mocks.h"
 
-CardDatabaseSettings::CardDatabaseSettings(QString settingPath, QObject *parent)
+CardDatabaseSettings::CardDatabaseSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "cardDatabase.ini", parent)
 {
 }

tests/carddatabase/mocks.cpp

@@ -1,7 +1,7 @@
 
 #include "mocks.h"
 
-CardDatabaseSettings::CardDatabaseSettings(QString settingPath, QObject *parent)
+CardDatabaseSettings::CardDatabaseSettings(const QString &settingPath, QObject *parent)
     : SettingsManager(settingPath + "cardDatabase.ini", parent)
 {
 }