commit 1357380105caf5cf0f62c2bdbf9172e8d40b0d6f
author Ed Tanous <edtanous@google.com> Mon Feb 28 21:58:46 2022 -0800
committer Ed Tanous <edtanous@google.com> Sat Mar 12 14:37:11 2022 -0800
tree ac149b3ad2a750ea295f3cf615ed21069a319702
parent eb6a9d0c7804b042e32ef97b5d4a0948b75222e3

Make getProperty mutable

The lambda passed through getProperty prevents passing mutable lambdas
through this interface.  For examples of why this is needed, there's a
bmcweb commit here that's attempting to replace templated callbacks with
non templated std::function, type erased types.

https://gerrit.openbmc-project.xyz/c/openbmc/bmcweb/+/51585

Tested: Above commit compiles.  Trivial change.

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

include/sdbusplus/asio/property.hpp

diff --git a/include/sdbusplus/asio/property.hpp b/include/sdbusplus/asio/property.hpp
index bbbd991..24c31bc 100644
--- a/include/sdbusplus/asio/property.hpp
+++ b/include/sdbusplus/asio/property.hpp
@@ -26,8 +26,9 @@
     static_assert(std::is_same_v<T, std::decay_t<T>>);
 
     bus.async_method_call(
-        [handler = std::move(handler)](boost::system::error_code ec,
-                                       std::variant<std::monostate, T>& ret) {
+        [handler =
+             std::move(handler)](boost::system::error_code ec,
+                                 std::variant<std::monostate, T>& ret) mutable {
             if (ec)
             {
                 handler(ec, {});