Searched hist:"2922 eebe4ddc565062eef84082519c094cb99f6e" (Results 1 – 2 of 2) sorted by relevance
/openbmc/phosphor-pid-control/dbus/ |
H A D | dbuspassiveredundancy.cpp | diff 2922eebe4ddc565062eef84082519c094cb99f6e Sun Dec 08 07:47:37 CST 2024 Jayanth Othayoth <ojayanth@gmail.com> clang-tidy: Fix Move Prevents Copy Elision
The following errors were reported during clang-tidy enablement due to moving temporary objects, which prevents copy elision. This fix addresses the issue by removing unnecessary std::move calls call, allowing the compiler to optimize the code through copy elision, resolving the error.
''' test/dbus_passive_unittest.cpp:613:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:71:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:833:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:877:18: error: moving a temporary object prevents copy elision [-Werror dbus/dbuspassiveredundancy.cpp:52:11: error: moving a temporary object prevents copy elision [-Werror '''
Tested: Build and unit tests passed successfully.
Change-Id: I320f048807085c3bbac0171a28c8505474aaf24a Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|
/openbmc/phosphor-pid-control/test/ |
H A D | dbus_passive_unittest.cpp | diff 2922eebe4ddc565062eef84082519c094cb99f6e Sun Dec 08 07:47:37 CST 2024 Jayanth Othayoth <ojayanth@gmail.com> clang-tidy: Fix Move Prevents Copy Elision
The following errors were reported during clang-tidy enablement due to moving temporary objects, which prevents copy elision. This fix addresses the issue by removing unnecessary std::move calls call, allowing the compiler to optimize the code through copy elision, resolving the error.
''' test/dbus_passive_unittest.cpp:613:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:71:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:833:18: error: moving a temporary object prevents copy elision [-Werror test/dbus_passive_unittest.cpp:877:18: error: moving a temporary object prevents copy elision [-Werror dbus/dbuspassiveredundancy.cpp:52:11: error: moving a temporary object prevents copy elision [-Werror '''
Tested: Build and unit tests passed successfully.
Change-Id: I320f048807085c3bbac0171a28c8505474aaf24a Signed-off-by: Jayanth Othayoth <ojayanth@gmail.com>
|