do not use auto and rename handles to providers

handles is a poor name choice for the list of IpmiProviders and being
declared as an auto is hard to see what type it really is. This changes
the type to be fixed and uses a more appropriate name for the variable.

Change-Id: I35f744c491f3810c7381738516b8ff0a6f20f129
Signed-off-by: Vernon Mauery <vernon.mauery@linux.intel.com>

1 files changed, 3 insertions, 2 deletions

diff --git a/ipmid-new.cpp b/ipmid-new.cpp
index ed45173..a92d4c4 100644
--- a/ipmid-new.cpp
+++ b/ipmid-new.cpp
@@ -599,7 +599,8 @@ int main(int argc, char* argv[])
     cmdManager = std::make_unique<phosphor::host::command::Manager>(*sdbusp);
 
     // Register all command providers and filters
-    auto handles = ipmi::loadProviders(HOST_IPMI_LIB_PATH);
+    std::forward_list<ipmi::IpmiProvider> providers =
+        ipmi::loadProviders(HOST_IPMI_LIB_PATH);
 
     // Add bindings for inbound IPMI requests
     auto server = sdbusplus::asio::object_server(sdbusp);
@@ -635,7 +636,7 @@ int main(int argc, char* argv[])
     ipmi::oemHandlerMap.clear();
     ipmi::filterList.clear();
     // unload the provider libraries
-    handles.clear();
+    providers.clear();
 
     return 0;
 }