Changeset 11016 for code/trunk/src/libraries/core/module/PluginManager.cc

Timestamp:

Jan 2, 2016, 4:06:30 PM (8 years ago)

Author:

landauf

Message:

Added config value to PluginManager to define whether a dereferenced plugin should be completely unloaded (i.e. the shared library is closed) or merely deactivated (i.e. the library remains in the process, but Identifiers and other framework components are removed).
This option is true by default on tardis machines because there it seems impossible to completely unload a shared library due to the use of STB_GNU_UNIQUE which prevents that dlclose() unloads the library. This caused errors when a plugin should have been reloaded, e.g. when the pong level was restarted.

File:

• code/trunk/src/libraries/core/module/PluginManager.cc (modified) (4 diffs)

code/trunk/src/libraries/core/module/PluginManager.cc

@@ -34 +34 @@
 #include "Plugin.h"
 #include "PluginReference.h"
+#include "core/CoreIncludes.h"
 #include "core/ApplicationPaths.h"
 #include "core/command/ConsoleCommandIncludes.h"
+#include "core/config/ConfigValueIncludes.h"
 #include "core/object/Context.h"
+
+#ifdef DO_NOT_UNLOAD_PLUGINS
+#  define MERELY_DEACTIVATE_PLUGINS true
+#else
+#  define MERELY_DEACTIVATE_PLUGINS false
+#endif
 
 namespace orxonox
@@ -48 +56 @@
     PluginManager* PluginManager::singletonPtr_s  = 0;
 
+    RegisterAbstractClass(PluginManager).inheritsFrom<Configurable>();
+
     PluginManager::PluginManager()
     {
+        RegisterObject(PluginManager);
+
         ModifyConsoleCommand("PluginManager", __CC_PluginManager_load_name).setObject(this);
         ModifyConsoleCommand("PluginManager", __CC_PluginManager_unload_name).setObject(this);
+
+        this->setConfigValues();
     }
 
@@ -63 +77 @@
         for (std::map<std::string, Plugin*>::iterator it = this->plugins_.begin(); it != this->plugins_.end(); ++it)
             delete it->second;
+    }
+
+    void PluginManager::setConfigValues()
+    {
+        SetConfigValue(bMerelyDeactivatePlugins_, MERELY_DEACTIVATE_PLUGINS);
     }
 
@@ -99 +118 @@
         Plugin* plugin = this->plugins_[name];
         if (plugin != NULL)
-            plugin->dereference(false);
+            plugin->dereference(this->bMerelyDeactivatePlugins_);
         else
             orxout(internal_warning) << "Cannot find plugin with name " << name << endl;