commit 8cc8edec45fb504457933c2c3f2625fa876493e6
author Ed Tanous <edtanous@google.com> Mon Feb 28 10:20:59 2022 -0800
committer Ed Tanous <ed@tanous.net> Mon Mar 07 18:04:33 2022 +0000
tree 86f7a8028f8b07dbb42a383828c0b456d4c7f4ee
parent 5b9ef70bd343047bb3c1f204b4a2de1030354704

Don't rely on operator << for object logging

In the upcoming fmt patch, we remove the use of streams, and a number of
our logging statements are relying on them.  This commit changes them to
no longer rely on operator>> or operator+ to build their strings.  This
alone isn't very useful, but in the context of the next patch makes the
automation able to do a complete conversion of all log statements
automatically.

Tested: enabled logging on local and saw log statements print to console

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

redfish-core/lib/managers.hpp

diff --git a/redfish-core/lib/managers.hpp b/redfish-core/lib/managers.hpp
index acaa40a..b0e2507 100644
--- a/redfish-core/lib/managers.hpp
+++ b/redfish-core/lib/managers.hpp
@@ -1872,8 +1872,8 @@
                 return;
             }
 
-            BMCWEB_LOG_DEBUG
-                << "Setting firmware version " + firmwareId + " to priority 0.";
+            BMCWEB_LOG_DEBUG << "Setting firmware version " << firmwareId
+                             << " to priority 0.";
 
             // Only support Immediate
             // An addition could be a Redfish Setting like