commit 0d7aca8ce24851f2206e7bf177469e2571179771
author George Liu <liuxiwei@inspur.com> Mon Mar 30 15:01:36 2020 +0800
committer Deepak Kodihalli <dkodihal@in.ibm.com> Mon May 11 02:57:40 2020 +0000
tree e59314a63cde62e6965a5a127210b9a6ba204f16
parent 456c9a2f7bc11ed3cef2139f7c1a316fa68fd2a6

pldm: Optimized the setXXXHandler method in platform.hpp

Since other PDR types(eg: numeric effecter PDR) need to be added later,
all setXXXHandler methods are best separated, which is easy to maintain
and read.
so moved the setStateEffecterStateHandler method to
platform_state_effecter.hpp.

Signed-off-by: George Liu <liuxiwei@inspur.com>
Change-Id: Iae120541963f3026e303f38f7960f51434be9de3

libpldmresponder/platform.cpp

diff --git a/libpldmresponder/platform.cpp b/libpldmresponder/platform.cpp
index 8d0cda0..df0f2d1 100644
--- a/libpldmresponder/platform.cpp
+++ b/libpldmresponder/platform.cpp
@@ -3,6 +3,7 @@
 
 #include "pdr_numeric_effecter.hpp"
 #include "pdr_state_effecter.hpp"
+#include "platform_state_effecter.hpp"
 #include "utils.hpp"
 
 namespace pldm
@@ -216,8 +217,9 @@
 
     stateField.resize(compEffecterCnt);
     const pldm::utils::DBusHandler dBusIntf;
-    rc = setStateEffecterStatesHandler<pldm::utils::DBusHandler>(
-        dBusIntf, effecterId, stateField);
+    rc = platform_state_effecter::setStateEffecterStatesHandler<
+        pldm::utils::DBusHandler, Handler>(dBusIntf, *this, effecterId,
+                                           stateField);
     if (rc != PLDM_SUCCESS)
     {
         return CmdHandler::ccOnlyResponse(request, rc);