Changeset 7966 for code/branches/usability/src/libraries/tools

Timestamp:

Feb 26, 2011, 5:00:17 PM (14 years ago)

Author:

landauf

Message:

some changes related to camera switching:

• added ViewportEventListener (currently only listens to camera changes in a viewport)
• Shader now correctly updates its visibility if the camera changes the scene
• (the same lines of code also fix the weird Ogre behavior which was originally fixed in CameraManager)
• GraphicsManager also updates the GUIManager's camera
• if all cameras are destroyed, CameraManager now officially switches to NULL camera

Location:

code/branches/usability/src/libraries/tools

Files:

• Shader.cc (modified) (4 diffs)
• Shader.h (modified) (3 diffs)

code/branches/usability/src/libraries/tools/Shader.cc

@@ -56 +56 @@
         this->bVisible_ = true;
         this->bLoadCompositor_ = GameMode::showsGraphics();
-        this->bViewportInitialized_ = false;
+        this->bViewportInitialized_ = true;
 
         if (this->bLoadCompositor_ && Ogre::Root::getSingletonPtr())
@@ -89 +89 @@
     {
         this->scenemanager_ = scenemanager;
-        this->bViewportInitialized_ = false;
-    }
-
+//        this->bViewportInitialized_ = false;
+    }
+
+    void Shader::cameraChanged(Ogre::Viewport* viewport, Ogre::Camera* oldCamera)
+    {
+        if (!this->scenemanager_ || (viewport != this->scenemanager_->getCurrentViewport() && this->scenemanager_->getCurrentViewport() != NULL))
+            return;
+
+        // update compositor in viewport (shader should only be active if the current camera is in the same scene as the shader)
+
+        // Note:
+        // The shader needs also to be switched off and on after changing the camera in the
+        // same scene to avoid weird behaviour with active compositors while switching the
+        // camera (like freezing the image)
+        //
+        // Last known Ogre version needing this workaround:
+        // 1.4.8
+        // 1.7.2
+
+
+        if (oldCamera && this->scenemanager_ == oldCamera->getSceneManager())
+            Ogre::CompositorManager::getSingleton().setCompositorEnabled(viewport, this->compositor_, false);
+
+        if (viewport->getCamera() && this->scenemanager_ == viewport->getCamera()->getSceneManager())
+            Ogre::CompositorManager::getSingleton().setCompositorEnabled(viewport, this->compositor_, this->isVisible());
+    }
+/*
     void Shader::tick(float dt)
     {
@@ -102 +126 @@
         }
     }
-
+*/
     void Shader::changedCompositor()
     {
@@ -119 +143 @@
                 if (!this->compositorInstance_)
                     COUT(2) << "Warning: Couldn't load compositor with name \"" << this->compositor_ << "\"." << std::endl;
-                Ogre::CompositorManager::getSingleton().setCompositorEnabled(viewport, this->compositor_, this->bViewportInitialized_ && this->isVisible());
+                //Ogre::CompositorManager::getSingleton().setCompositorEnabled(viewport, this->compositor_, this->bViewportInitialized_ && this->isVisible());
             }
             this->oldcompositor_ = this->compositor_;

code/branches/usability/src/libraries/tools/Shader.h

@@ -37 +37 @@
 
 #include "util/OgreForwardRefs.h"
-#include "tools/interfaces/Tickable.h"
+#include "core/ViewportEventListener.h"
 
 namespace orxonox
 {
-    class _ToolsExport Shader : public Tickable
+    class _ToolsExport Shader : public ViewportEventListener
     {
         typedef std::pair<bool, void*>                  ParameterPointer;
@@ -52 +52 @@
             Shader(Ogre::SceneManager* scenemanager = 0);
             virtual ~Shader();
-
-            virtual void tick(float dt);
 
             inline void setVisible(bool bVisible)
@@ -83 +81 @@
                 { return this->scenemanager_; }
 
+            virtual void cameraChanged(Ogre::Viewport* viewport, Ogre::Camera* oldCamera);
+
             void setParameter(const std::string& material, size_t technique, size_t pass, const std::string& parameter, float value);
             void setParameter(const std::string& material, size_t technique, size_t pass, const std::string& parameter, int value);