Cppcheck: CWE: 398: Parameter 'textValue' can be declared with const

diff --git a/Src/Common/IniOptionsMgr.cpp b/Src/Common/IniOptionsMgr.cpp
index dbbf234..eed8171 100644 (file)
--- a/Src/Common/IniOptionsMgr.cpp
+++ b/Src/Common/IniOptionsMgr.cpp
@@ -75,7 +75,7 @@ std::map<String, String> CIniOptionsMgr::Load(const String& iniFilePath)
        return iniFileKeyValues;
 }
 
-int CIniOptionsMgr::LoadValueFromBuf(const String& strName, String& textValue, varprop::VariantValue& value)
+int CIniOptionsMgr::LoadValueFromBuf(const String& strName, const String& textValue, varprop::VariantValue& value)
 {
        int retVal = COption::OPT_OK;
        int valType = value.GetType();

diff --git a/Src/Common/IniOptionsMgr.h b/Src/Common/IniOptionsMgr.h
index 6cafbd2..bb85c97 100644 (file)
--- a/Src/Common/IniOptionsMgr.h
+++ b/Src/Common/IniOptionsMgr.h
@@ -39,7 +39,7 @@ public:
 
 protected:
        static std::map<String, String> Load(const String& iniFilePath);
-       int LoadValueFromBuf(const String& strName, String& textValue, varprop::VariantValue& value);
+       int LoadValueFromBuf(const String& strName, const String& textValue, varprop::VariantValue& value);
        const TCHAR *GetFilePath() const { return m_filePath.c_str(); }
        int SaveValueToFile(const String& strValueName,
                const varprop::VariantValue& value);