commit f8fe53e7114ab10c9059377541277739ace5c1ff
author Ed Tanous <edtanous@google.com> Thu Jun 30 15:55:45 2022 -0700
committer Ed Tanous <ed@tanous.net> Wed Dec 21 17:15:07 2022 +0000
tree 368784d5e265e876e05523493cf058c5d24c8240
parent 6dcbb8675d5c5765c3e53354b8f3b339effecd44

Change variable scopes

cppcheck correctly notes that a lot of our variables can be declared at
more specific scopes, and in every case, it seems to be correct.

Tested: Redfish service validator passes.  Unit test coverage on others.

Signed-off-by: Ed Tanous <edtanous@google.com>
Change-Id: Ia4414410d0e8f74a3bd40fdc0e0232450d1a6416

http/http_response.hpp

diff --git a/http/http_response.hpp b/http/http_response.hpp
index c1f25a5..b1b15fb 100644
--- a/http/http_response.hpp
+++ b/http/http_response.hpp
@@ -41,9 +41,9 @@
     {}
 
     Response(Response&& res) noexcept :
-        stringResponse(std::move(res.stringResponse)), completed(res.completed)
+        stringResponse(std::move(res.stringResponse)),
+        jsonValue(std::move(res.jsonValue)), completed(res.completed)
     {
-        jsonValue = std::move(res.jsonValue);
         // See note in operator= move handler for why this is needed.
         if (!res.completed)
         {