Fixes to sceneless renderpath on OpenGL.

Source/Engine/Graphics/Renderer.cpp

@@ -587,27 +587,26 @@ void Renderer::Update(float timeStep)
         
         const IntRect& viewRect = viewport->GetRect();
         Scene* scene = viewport->GetScene();
-        if (!scene)
-            continue;
-        
-        Octree* octree = scene->GetComponent<Octree>();
-        
-        // Update octree (perform early update for drawables which need that, and reinsert moved drawables.)
-        // However, if the same scene is viewed from multiple cameras, update the octree only once
-        if (!updatedOctrees_.Contains(octree))
+        Octree* octree = scene ? scene->GetComponent<Octree>() : 0;
+        if (scene && octree)
         {
-            frame_.camera_ = viewport->GetCamera();
-            frame_.viewSize_ = viewRect.Size();
-            if (frame_.viewSize_ == IntVector2::ZERO)
-                frame_.viewSize_ = IntVector2(graphics_->GetWidth(), graphics_->GetHeight());
-            octree->Update(frame_);
-            updatedOctrees_.Insert(octree);
-            
-            // Set also the view for the debug renderer already here, so that it can use culling
-            /// \todo May result in incorrect debug geometry culling if the same scene is drawn from multiple viewports
-            DebugRenderer* debug = scene->GetComponent<DebugRenderer>();
-            if (debug)
-                debug->SetView(viewport->GetCamera());
+            // Update octree (perform early update for drawables which need that, and reinsert moved drawables.)
+            // However, if the same scene is viewed from multiple cameras, update the octree only once
+            if (!updatedOctrees_.Contains(octree))
+            {
+                frame_.camera_ = viewport->GetCamera();
+                frame_.viewSize_ = viewRect.Size();
+                if (frame_.viewSize_ == IntVector2::ZERO)
+                    frame_.viewSize_ = IntVector2(graphics_->GetWidth(), graphics_->GetHeight());
+                octree->Update(frame_);
+                updatedOctrees_.Insert(octree);
+                
+                // Set also the view for the debug renderer already here, so that it can use culling
+                /// \todo May result in incorrect debug geometry culling if the same scene is drawn from multiple viewports
+                DebugRenderer* debug = scene->GetComponent<DebugRenderer>();
+                if (debug)
+                    debug->SetView(viewport->GetCamera());
+            }
         }
         
         // Update view. This may queue further views

Source/Engine/Graphics/View.cpp

@@ -315,6 +315,7 @@ bool View::Define(RenderSurface* renderTarget, Viewport* viewport)
         return false;
     
     hasScenePasses_ = false;
+    flipVertical_ = false;
     
     // Make sure that all necessary batch queues exist
     scenePasses_.Clear();
@@ -399,6 +400,13 @@ bool View::Define(RenderSurface* renderTarget, Viewport* viewport)
     litBasePassName_ = PASS_LITBASE;
     litAlphaPassName_ = PASS_LITALPHA;
     
+    // On OpenGL, flip the projection if rendering to a texture so that the texture can be addressed in the same way
+    // as a render texture produced on Direct3D9
+    #ifdef USE_OPENGL
+    if (renderTarget_)
+        flipVertical_ = true;
+    #endif
+    
     // Go through commands to check for deferred rendering and other flags
     deferred_ = false;
     deferredAmbient_ = false;
@@ -494,12 +502,12 @@ void View::Update(const FrameInfo& frame)
     for (HashMap<StringHash, BatchQueue>::Iterator i = batchQueues_.Begin(); i != batchQueues_.End(); ++i)
         i->second_.Clear(maxSortedInstances);
     
-    if (hasScenePasses_ && (!camera_ || !octree_))
-        return;
-    
-    // Set automatic aspect ratio if required
-    if (camera_->GetAutoAspectRatio())
-        camera_->SetAspectRatioInternal((float)frame_.viewSize_.x_ / (float)frame_.viewSize_.y_);
+    if (camera_)
+    {
+        // Set automatic aspect ratio if required
+        if (camera_->GetAutoAspectRatio())
+            camera_->SetAspectRatioInternal((float)frame_.viewSize_.x_ / (float)frame_.viewSize_.y_);
+    }
     
     GetDrawables();
     GetBatches();
@@ -507,9 +515,6 @@ void View::Update(const FrameInfo& frame)
 
 void View::Render()
 {
-    if (hasScenePasses_ && (!octree_ || !camera_))
-        return;
-    
     // Actually update geometry data now
     UpdateGeometries();
     
@@ -541,20 +546,17 @@ void View::Render()
     }
     #endif
     
-    if (renderTarget_)
-    {
-        // On OpenGL, flip the projection if rendering to a texture so that the texture can be addressed in the same way
-        // as a render texture produced on Direct3D9
-        #ifdef USE_OPENGL
-        camera_->SetFlipVertical(true);
-        #endif
-    }
+    #ifdef USE_OPENGL
+    if (camera_)
+        camera_->SetFlipVertical(flipVertical_);
+    #endif
     
     // Render
     ExecuteRenderPathCommands();
     
     #ifdef USE_OPENGL
-    camera_->SetFlipVertical(false);
+    if (camera_)
+        camera_->SetFlipVertical(false);
     #endif
     
     graphics_->SetDepthBias(0.0f, 0.0f);
@@ -567,7 +569,7 @@ void View::Render()
         BlitFramebuffer(static_cast<Texture2D*>(currentRenderTarget_->GetParentTexture()), renderTarget_, true);
     
     // If this is a main view, draw the associated debug geometry now
-    if (!renderTarget_ && octree_)
+    if (!renderTarget_ && camera_ && octree_)
     {
         DebugRenderer* debug = octree_->GetComponent<DebugRenderer>();
         if (debug && debug->IsEnabledEffective())
@@ -599,6 +601,9 @@ Renderer* View::GetRenderer() const
 
 void View::GetDrawables()
 {
+    if (!camera_ || !octree_)
+        return;
+    
     PROFILE(GetDrawables);
     
     WorkQueue* queue = GetSubsystem<WorkQueue>();
@@ -765,6 +770,9 @@ void View::GetDrawables()
 
 void View::GetBatches()
 {
+    if (!camera_ || !octree_)
+        return;
+    
     WorkQueue* queue = GetSubsystem<WorkQueue>();
     PODVector<Light*> vertexLights;
     BatchQueue* alphaQueue = batchQueues_.Contains(alphaPassName_) ? &batchQueues_[alphaPassName_] : (BatchQueue*)0;
@@ -1872,7 +1880,7 @@ void View::DrawFullscreenQuad(bool nearQuad)
     Matrix4 projection = Matrix4::IDENTITY;
     
     #ifdef USE_OPENGL
-    if (camera_->GetFlipVertical())
+    if (flipVertical_)
         projection.m11_ = -1.0f;
     model.m23_ = nearQuad ? -1.0f : 1.0f;
     #else

Source/Engine/Graphics/View.h

@@ -306,6 +306,8 @@ private:
     bool useLitBase_;
     /// Has scene passes flag. If no scene passes, view can be defined without a valid scene or camera to only perform quad rendering.
     bool hasScenePasses_;
+    /// Flip vertical flag.
+    bool flipVertical_;
     /// Renderpath.
     RenderPath* renderPath_;
     /// Per-thread octree query results.