Merge pull request #3115 from pljones/options/pass-commandline-by-ref…

…erence Refactor: Pass "CommandLineOptions" to CSettings::Load by reference
jamulussoftware · Sep 27, 2023 · a9b5475 · a9b5475
2 parents f538b22 + 18e235e
commit a9b5475
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/src/settings.cpp b/src/settings.cpp
@@ -25,7 +25,7 @@
 #include "settings.h"
 
 /* Implementation *************************************************************/
-void CSettings::Load ( const QList<QString> CommandLineOptions )
+void CSettings::Load ( const QList<QString>& CommandLineOptions )
 {
     // prepare file name for loading initialization data from XML file and read
     // data from file if possible

diff --git a/src/settings.h b/src/settings.h
@@ -52,7 +52,7 @@ class CSettings : public QObject
         QObject::connect ( QCoreApplication::instance(), &QCoreApplication::aboutToQuit, this, &CSettings::OnAboutToQuit );
     }
 
-    void Load ( const QList<QString> CommandLineOptions );
+    void Load ( const QList<QString>& CommandLineOptions );
     void Save();
 
     // common settings