Changeset 1646 for code/branches/gui/src/orxonox/gui/GUIManager.cc

Timestamp:

Jul 24, 2008, 10:40:22 PM (16 years ago)

Author:

rgrieder

Message:

• privatised InputState c'tors
• added destruction code for GUIManager
• fixed some issues and bugs
• found 2400 memory leaks
• haven't done anything about it
• converted GUIManager to Ogre Singleton
• added NULL checkers in Loader

File:

• code/branches/gui/src/orxonox/gui/GUIManager.cc (modified) (4 diffs)

code/branches/gui/src/orxonox/gui/GUIManager.cc

@@ -58 +58 @@
     SetConsoleCommandShortcut(GUIManager, showGUI_s).setKeybindMode(KeybindMode::OnPress);
 
+    GUIManager* GUIManager::singletonRef_s = 0;
+
     GUIManager::GUIManager()
         //: emptySceneManager_(0)
@@ -71 +73 @@
         , state_(Uninitialised)
     {
+        assert(singletonRef_s == 0);
+        singletonRef_s = this;
     }
 
     GUIManager::~GUIManager()
     {
-        // TODO: destruct at least something
+        if (backgroundCamera_)
+            backgroundSceneManager_->destroyCamera(backgroundCamera_);
+
+        if (backgroundSceneManager_)
+            Ogre::Root::getSingleton().destroySceneManager(backgroundSceneManager_);
+
+        InputManager::getInstance().destroyState("gui");
+
+        if (guiSystem_)
+            delete guiSystem_;
+
+        if (scriptModule_)
+        {
+            // destroy our own tolua interfaces
+                //lua_pushnil(luaState_);
+                //lua_setglobal(luaState_, "Orxonox");
+                //lua_pushnil(luaState_);
+                //lua_setglobal(luaState_, "Core");
+            // TODO: deleting the script module fails an assertation.
+            // However there is not much we can do about it since it occurs too when
+            // we don't open Core or Orxonox. Might be a CEGUI issue.
+            // The memory leak is not a problem anyway..
+            //delete scriptModule_;
+        }
+
+        if (guiRenderer_)
+            delete guiRenderer_;
+
+        singletonRef_s = 0;
     }
 
@@ -104 +136 @@
                 // setup scripting
                 this->scriptModule_ = new LuaScriptModule();
+                this->luaState_ = this->scriptModule_->getLuaState();
 
                 // create the CEGUI system singleton
@@ -267 +300 @@
 
 
-    /**
-    @brief
-        Returns a unique instance of GUIManager.
-    @return
-        The instance
-    */
-    GUIManager& GUIManager::getInstance()
-    {
-        static GUIManager instance;
-        return instance;
-    }
-
     inline CEGUI::MouseButton GUIManager::convertButton(MouseButton::Enum button)
     {