commit 81c4e3305281b2cfece8822f7c7114200ce4e12c
author Ed Tanous <edtanous@google.com> Thu May 18 10:30:34 2023 -0700
committer Ed Tanous <ed@tanous.net> Fri May 19 17:47:39 2023 +0000
tree 60563dba369c24a57bbcc21f6a172490e338a3f2
parent 4a40441f944aaff8d7bb2040e4d88ebfcebbab11

Capture all boost::system::error_codes by ref

Capturing these possibly overloaded values by reference can avoid a copy
in some cases, and it's good to be consistent.

This change was made automatically by grep/sed.

Tested: Code compiles.

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

include/obmc_console.hpp

diff --git a/include/obmc_console.hpp b/include/obmc_console.hpp
index 3ab13ee..fac489a 100644
--- a/include/obmc_console.hpp
+++ b/include/obmc_console.hpp
@@ -50,7 +50,7 @@
     doingWrite = true;
     hostSocket->async_write_some(
         boost::asio::buffer(inputBuffer.data(), inputBuffer.size()),
-        [](boost::beast::error_code ec, std::size_t bytesWritten) {
+        [](const boost::beast::error_code& ec, std::size_t bytesWritten) {
         doingWrite = false;
         inputBuffer.erase(0, bytesWritten);
 

include/vm_websocket.hpp

diff --git a/include/vm_websocket.hpp b/include/vm_websocket.hpp
index 89270d9..fed3681 100644
--- a/include/vm_websocket.hpp
+++ b/include/vm_websocket.hpp
@@ -90,7 +90,7 @@
         doingWrite = true;
         pipeIn.async_write_some(
             inputBuffer->data(),
-            [this, self(shared_from_this())](boost::beast::error_code ec,
+            [this, self(shared_from_this())](const boost::beast::error_code& ec,
                                              std::size_t bytesWritten) {
             BMCWEB_LOG_DEBUG << "Wrote " << bytesWritten << "bytes";
             doingWrite = false;