clang-tidy: Enable readability-static-accessed-through-instance check
Checks for member expressions that access static members through
instances, and replaces them with uses of the appropriate
qualified-id.
Signed-off-by: George Liu <liuxiwei@ieisystem.com>
Change-Id: I24de8ac8897de839594c6c71319febd0209bbc2e
diff --git a/manager/json-config.hpp b/manager/json-config.hpp
index b4a4efa..7d94f20 100644
--- a/manager/json-config.hpp
+++ b/manager/json-config.hpp
@@ -159,14 +159,14 @@
try
{
// Get all objects implementing the compatible interface
- auto objects =
- dBusHandler.getSubTreePaths("/", confCompatibleInterface);
+ auto objects = phosphor::led::utils::DBusHandler::getSubTreePaths(
+ "/", confCompatibleInterface);
for (const auto& path : objects)
{
try
{
// Retrieve json config compatible relative path locations
- auto value = dBusHandler.getProperty(
+ auto value = phosphor::led::utils::DBusHandler::getProperty(
path, confCompatibleInterface, confCompatibleProperty);
auto confCompatValues =
@@ -218,9 +218,6 @@
* interface to show up.
*/
std::unique_ptr<sdbusplus::bus::match_t> match;
-
- /** DBusHandler class handles the D-Bus operations */
- utils::DBusHandler dBusHandler;
};
/** Blocking call to find the JSON Config from DBus. */
diff --git a/manager/led-main.cpp b/manager/led-main.cpp
index 7791747..f671d5d 100644
--- a/manager/led-main.cpp
+++ b/manager/led-main.cpp
@@ -66,7 +66,7 @@
phosphor::led::LampTest lampTest(event, manager);
// Clear leds triggered by lamp test in previous boot
- lampTest.clearLamps();
+ phosphor::led::LampTest::clearLamps();
groups.emplace_back(std::make_unique<phosphor::led::Group>(
bus, LAMP_TEST_OBJECT, manager, serializePtr,