fix unload warning (#274)

* fix unload warning

Signed-off-by: Chen Lihui <[email protected]>

* adjust order for header file

Signed-off-by: Chen Lihui <[email protected]>

---------

Signed-off-by: Chen Lihui <[email protected]>

Author: Chen Lihui

qt_gui_cpp/include/qt_gui_cpp/recursive_plugin_provider.h

@@ -36,6 +36,7 @@
 #include "composite_plugin_provider.h"
 #include "ros_pluginlib_plugin_provider_for_plugin_providers.h"
 
+#include <QList>
 #include <QMap>
 #include <QString>
 
@@ -54,9 +55,12 @@ class RecursivePluginProvider
 
   virtual QMap<QString, QString> discover(QObject* discovery_data);
 
+  virtual void shutdown();
+
 private:
 
   RosPluginlibPluginProvider_ForPluginProviders* plugin_provider_;
+  QList<PluginProvider*> providers_;
 
 };
 

qt_gui_cpp/src/qt_gui_cpp/recursive_plugin_provider.cpp

@@ -59,7 +59,6 @@ QMap<QString, QString> RecursivePluginProvider::discover(QObject* discovery_data
   }
 
   // instantiate plugins
-  QList<PluginProvider*> providers;
   for (QList<QString>::iterator it = plugin_ids.begin(); it != plugin_ids.end(); it++)
   {
     try
@@ -70,7 +69,7 @@ QMap<QString, QString> RecursivePluginProvider::discover(QObject* discovery_data
       {
         throw std::runtime_error("load returned None");
       }
-      providers.append(instance);
+      providers_.append(instance);
     }
     catch (...)
     {
@@ -79,8 +78,17 @@ QMap<QString, QString> RecursivePluginProvider::discover(QObject* discovery_data
   }
 
   // delegate discovery through instantiated plugin providers to base class
-  set_plugin_providers(providers);
+  set_plugin_providers(providers_);
   return CompositePluginProvider::discover(discovery_data);
 }
 
+void RecursivePluginProvider::shutdown()
+{
+  for (QList<PluginProvider*>::iterator it = providers_.begin(); it != providers_.end(); it++)
+  {
+    plugin_provider_->unload(*it);
+  }
+  CompositePluginProvider::shutdown();
+}
+
 } // namespace

qt_gui_cpp/src/qt_gui_cpp_sip/recursive_plugin_provider.sip

@@ -16,6 +16,7 @@ public:
 
   virtual QMap<QString, QString> discover(QObject* discovery_data);
 
+  virtual void shutdown();
 };
 
 };