diff options
| author | Ed Tanous <edtanous@google.com> | 2022-02-07 20:35:31 +0300 |
|---|---|---|
| committer | Ed Tanous <ed@tanous.net> | 2022-02-08 22:11:52 +0300 |
| commit | 331b2017f90ba0a043c332aab552e09008682143 (patch) | |
| tree | 88bd119e810fadeb76dc2ef33060521dac58f42b /redfish-core/src/error_messages.cpp | |
| parent | ba27e59e5049d4ae12baf6dfd84d3045dd1e55c9 (diff) | |
| download | bmcweb-331b2017f90ba0a043c332aab552e09008682143.tar.xz | |
Capture int by value in stringValueTooLong
Per the coding standard, we should be capturing this by int, not const
int&.
Tested: There are no uses of stringValueTooLong(), so noop change to
binary.
Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Idfcb05c962e0a0d489db263a6f845bb1789b5842
Diffstat (limited to 'redfish-core/src/error_messages.cpp')
| -rw-r--r-- | redfish-core/src/error_messages.cpp | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/redfish-core/src/error_messages.cpp b/redfish-core/src/error_messages.cpp index 83ebbfedec..599eeae3cd 100644 --- a/redfish-core/src/error_messages.cpp +++ b/redfish-core/src/error_messages.cpp @@ -696,7 +696,7 @@ void actionParameterMissing(crow::Response& res, const std::string& arg1, * See header file for more information * @endinternal */ -nlohmann::json stringValueTooLong(const std::string& arg1, const int& arg2) +nlohmann::json stringValueTooLong(const std::string& arg1, int arg2) { return nlohmann::json{ {"@odata.type", "#Message.v1_1_1.Message"}, @@ -709,8 +709,7 @@ nlohmann::json stringValueTooLong(const std::string& arg1, const int& arg2) "Resubmit the request with an appropriate string length."}}; } -void stringValueTooLong(crow::Response& res, const std::string& arg1, - const int& arg2) +void stringValueTooLong(crow::Response& res, const std::string& arg1, int arg2) { res.result(boost::beast::http::status::bad_request); addMessageToErrorJson(res.jsonValue, stringValueTooLong(arg1, arg2)); |