errmsg: Added message 'function parameter is passed by value'

src/checkother.cpp

@@ -619,9 +619,7 @@ void CheckOther::CheckConstantFunctionParameter()
     {
         if (Token::Match(tok, "[,(] const std :: %type% %var% [,)]"))
         {
-            std::ostringstream errmsg;
+            _errorLogger->reportErr(ErrorMessage::passedByValue(_tokenizer, tok, tok->strAt(5)));
-            errmsg << _tokenizer->fileLine(tok) << " " << tok->strAt(5) << " is passed by value, it could be passed by reference/pointer instead";
-            _errorLogger->reportErr(errmsg.str());
         }
 
         else if (Token::Match(tok, "[,(] const %type% %var% [,)]"))
@@ -630,9 +628,7 @@ void CheckOther::CheckConstantFunctionParameter()
             const std::string pattern(std::string("class|struct ") + tok->strAt(2));
             if (Token::findmatch(_tokenizer->tokens(), pattern.c_str()))
             {
-                std::ostringstream errmsg;
+                _errorLogger->reportErr(ErrorMessage::passedByValue(_tokenizer, tok, tok->strAt(3)));
-                errmsg << _tokenizer->fileLine(tok) << " " << tok->strAt(3) << " is passed by value, it could be passed by reference/pointer instead";
-                _errorLogger->reportErr(errmsg.str());
             }
         }
     }

src/cppcheck.cpp

@@ -327,6 +327,9 @@ void CppCheck::checkFile(const std::string &code, const char FileName[])
     if (ErrorMessage::unreachableCode(_settings))
         checkOther.unreachableCode();
 
+    // Check if a constant function parameter is passed by value
+    if (ErrorMessage::passedByValue(_settings))
+        checkOther.CheckConstantFunctionParameter();
 
 
     if (_settings._checkCodingStyle)
@@ -334,9 +337,6 @@ void CppCheck::checkFile(const std::string &code, const char FileName[])
         // Variable scope (check if the scope could be limited)
         //CheckVariableScope();
 
-        // Check if a constant function parameter is passed by value
-        checkOther.CheckConstantFunctionParameter();
-
         // Check for various types of incomplete statements that could for example
         // mean that an ';' has been added by accident
         checkOther.CheckIncompleteStatement();

src/errormessage.h

@@ -237,5 +237,14 @@ public:
         return true;
     }
 
+    static std::string passedByValue(const Tokenizer *tokenizer, const Token *Location, const std::string &parname)
+    {
+        return msg1(tokenizer, Location) + "Function parameter '" + parname + "' is passed by value. It could be passed by reference instead.";
+    }
+    static bool passedByValue(const Settings &s)
+    {
+        return true;
+    }
+
 };
 #endif

tools/errmsg.cpp

@@ -85,6 +85,7 @@ int main()
     err.push_back(Message("udivWarning", Message::STYLE | Message::ALL, "Warning: Division with signed and unsigned operators"));
     err.push_back(Message("unusedStructMember", Message::STYLE, "struct or union member '%1::%2' is never used", "structname", "varname"));
     err.push_back(Message("unreachableCode", 0, "Unreachable code below a 'return'"));
+    err.push_back(Message("passedByValue", 0, "Function parameter '%1' is passed by value. It could be passed by reference instead.", "parname"));
 
     // Generate code..
     std::cout << "Generate code.." << std::endl;