From 0d12fb0346b016f4207a4feaf37f6fe9767c48ac Mon Sep 17 00:00:00 2001
From: Anatoly Baksheev <no@email>
Date: Sun, 19 Jan 2014 01:55:40 +0400
Subject: [PATCH] implemented workaround of a problem with x-server for ubuntu

---
 modules/viz/src/precomp.hpp |  1 +
 modules/viz/src/vizimpl.cpp | 41 +++++++++++++++++++++++++++++------------
 modules/viz/src/vizimpl.hpp |  2 ++
 3 files changed, 32 insertions(+), 12 deletions(-)

diff --git a/modules/viz/src/precomp.hpp b/modules/viz/src/precomp.hpp
index 7440f1e..de5346e 100644
--- a/modules/viz/src/precomp.hpp
+++ b/modules/viz/src/precomp.hpp
@@ -51,6 +51,7 @@
 #include <list>
 #include <vector>
 #include <iomanip>
+#include <limits>
 
 #include <vtkAppendPolyData.h>
 #include <vtkAssemblyPath.h>
diff --git a/modules/viz/src/vizimpl.cpp b/modules/viz/src/vizimpl.cpp
index e50666e..64222b4 100644
--- a/modules/viz/src/vizimpl.cpp
+++ b/modules/viz/src/vizimpl.cpp
@@ -45,24 +45,18 @@
 
 #include "precomp.hpp"
 
+
 /////////////////////////////////////////////////////////////////////////////////////////////
-cv::viz::Viz3d::VizImpl::VizImpl(const String &name) : spin_once_state_(false), widget_actor_map_(new WidgetActorMap)
+cv::viz::Viz3d::VizImpl::VizImpl(const String &name) : spin_once_state_(false),
+    window_position_(Vec2i(std::numeric_limits<int>::min())), widget_actor_map_(new WidgetActorMap)
 {
     renderer_ = vtkSmartPointer<vtkRenderer>::New();
+    window_name_ = VizStorage::generateWindowName(name);
 
     // Create render window
     window_ = vtkSmartPointer<vtkRenderWindow>::New();
-    window_name_ = VizStorage::generateWindowName(name);
-
     cv::Vec2i window_size = cv::Vec2i(window_->GetScreenSize()) / 2;
     window_->SetSize(window_size.val);
-    window_->AddRenderer(renderer_);
-    window_->AlphaBitPlanesOff();
-    window_->PointSmoothingOff();
-    window_->LineSmoothingOff();
-    window_->PolygonSmoothingOff();
-    window_->SwapBuffersOn();
-    window_->SetStereoTypeToAnaglyph();
 
     // Create the interactor style
     style_ = vtkSmartPointer<InteractorStyle>::New();
@@ -111,9 +105,32 @@ void cv::viz::Viz3d::VizImpl::close()
     interactor_->TerminateApp(); // This tends to close the window...
 }
 
+void cv::viz::Viz3d::VizImpl::recreateRenderWindow()
+{
+    //recreating is workaround for Ubuntu -- a crash in x-server
+    Vec2i window_size(window_->GetSize());
+    int fullscreen = window_->GetFullScreen();
+
+    window_ = vtkSmartPointer<vtkRenderWindow>::New();
+    if (window_position_[0] != std::numeric_limits<int>::min()) //also workaround
+        window_->SetPosition(window_position_.val);
+
+    window_->SetSize(window_size.val);
+    window_->SetFullScreen(fullscreen);
+    window_->AddRenderer(renderer_);
+    window_->AlphaBitPlanesOff();
+    window_->PointSmoothingOff();
+    window_->LineSmoothingOff();
+    window_->PolygonSmoothingOff();
+    window_->SwapBuffersOn();
+    window_->SetStereoTypeToAnaglyph();
+}
+
+
 /////////////////////////////////////////////////////////////////////////////////////////////
 void cv::viz::Viz3d::VizImpl::spin()
 {
+    recreateRenderWindow();
     interactor_ = vtkSmartPointer<vtkRenderWindowInteractor>::New();
     interactor_->SetRenderWindow(window_);
     interactor_->SetInteractorStyle(style_);
@@ -129,6 +146,7 @@ void cv::viz::Viz3d::VizImpl::spinOnce(int time, bool force_redraw)
     if (interactor_ == 0)
     {
         spin_once_state_ = true;
+        recreateRenderWindow();
         interactor_ = vtkSmartPointer<vtkRenderWindowInteractor>::New();
         interactor_->SetRenderWindow(window_);
         interactor_->SetInteractorStyle(style_);
@@ -400,7 +418,6 @@ void cv::viz::Viz3d::VizImpl::setViewerPose(const Affine3d &pose)
     camera.SetViewUp(up_vec.val);
 
     renderer_->ResetCameraClippingRange();
-    renderer_->Render();
 }
 
 /////////////////////////////////////////////////////////////////////////////////////////////
@@ -510,6 +527,6 @@ void cv::viz::Viz3d::VizImpl::setRepresentation(int representation)
 //////////////////////////////////////////////////////////////////////////////////////////////
 cv::String cv::viz::Viz3d::VizImpl::getWindowName() const { return window_name_; }
 void cv::viz::Viz3d::VizImpl::setFullScreen(bool mode) { window_->SetFullScreen(mode); }
-void cv::viz::Viz3d::VizImpl::setWindowPosition(const Point& position) { window_->SetPosition(position.x, position.y); }
+void cv::viz::Viz3d::VizImpl::setWindowPosition(const Point& position) { window_position_ = position; window_->SetPosition(position.x, position.y); }
 void cv::viz::Viz3d::VizImpl::setWindowSize(const Size& window_size) { window_->SetSize(window_size.width, window_size.height); }
 cv::Size cv::viz::Viz3d::VizImpl::getWindowSize() const { return Size(Point(Vec2i(window_->GetSize()))); }
diff --git a/modules/viz/src/vizimpl.hpp b/modules/viz/src/vizimpl.hpp
index 11e22ee..9eb918a 100644
--- a/modules/viz/src/vizimpl.hpp
+++ b/modules/viz/src/vizimpl.hpp
@@ -122,6 +122,7 @@ private:
 
     vtkSmartPointer<vtkRenderWindow> window_;
     String window_name_;
+    Vec2i window_position_;
 
     vtkSmartPointer<TimerCallback> timer_callback_;
     vtkSmartPointer<ExitCallback> exit_callback_;
@@ -131,6 +132,7 @@ private:
     Ptr<WidgetActorMap> widget_actor_map_;
 
     bool removeActorFromRenderer(vtkSmartPointer<vtkProp> actor);
+    void recreateRenderWindow();
 };
 
 #endif
-- 
2.7.4