diff options
author | Ed Tanous <ed@tanous.net> | 2024-04-07 23:52:22 +0300 |
---|---|---|
committer | Ed Tanous <ed@tanous.net> | 2024-04-11 21:21:49 +0300 |
commit | 8c3faccc941d5b32722554f8f24f320fd02dd4ab (patch) | |
tree | bc18d2f57bed20f906a1aa2378075855626f2365 /redfish-core | |
parent | c98dbc641e0973e8db4f701c18d3254f40a02e7b (diff) | |
download | bmcweb-8c3faccc941d5b32722554f8f24f320fd02dd4ab.tar.xz |
Remove logically dead code
This code path is subtle, but given that slotPresent is only set to true
if totalCores is incremented, there's no way to actually hit this
section of code.
Looking for input on if this is the right behavior.
Change-Id: Ie6dadd2c7a0ca6b8402148ddd9b8a369a4a38b2e
Signed-off-by: Ed Tanous <ed@tanous.net>
Diffstat (limited to 'redfish-core')
-rw-r--r-- | redfish-core/lib/processor.hpp | 6 |
1 files changed, 0 insertions, 6 deletions
diff --git a/redfish-core/lib/processor.hpp b/redfish-core/lib/processor.hpp index 5c229f4f9f..7b22c86d2a 100644 --- a/redfish-core/lib/processor.hpp +++ b/redfish-core/lib/processor.hpp @@ -282,12 +282,6 @@ inline void getCpuDataByService(std::shared_ptr<bmcweb::AsyncResp> asyncResp, // count is zero, then it has a higher precedence. if (slotPresent) { - if (totalCores == 0) - { - // Slot is not populated, set status end return - asyncResp->res.jsonValue["Status"]["State"] = "Absent"; - asyncResp->res.jsonValue["Status"]["Health"] = "OK"; - } asyncResp->res.jsonValue["TotalCores"] = totalCores; } return; |