0024358: TKV3d - connected structures are not re-computed on device lost

Graphic3d_StructureManager::ReComputeStructures() method was corrected to consider all child (connected) structures.
Corrected remarks: added image dump in test case; corrected method name.

diff --git a/src/Graphic3d/Graphic3d_StructureManager.cdl b/src/Graphic3d/Graphic3d_StructureManager.cdl
index 187548bfad8a9d8b505e7322e234e0c23fd40fa3..14652c3dce0be6f9835d0c3863e18035ef6d858e 100644 (file)
--- a/src/Graphic3d/Graphic3d_StructureManager.cdl
+++ b/src/Graphic3d/Graphic3d_StructureManager.cdl
@@ -459,10 +459,15 @@ is
        ---Purpose: Suppress the highlighting on the structure <AStructure>.
        ---Category: Private methods
 
-  ReComputeStructures (me: mutable);
+  RecomputeStructures (me: mutable);
   -- Purpose: Recomputes all displayed structures. Used to recompute GL
   -- resources after the last view has been closed without removing objects.
 
+  RecomputeStructures (me: mutable;
+                       theStructures : MapOfStructure from Graphic3d)
+  is private;
+  ---Purpose: Recomputes all structures from theStructures.
+
 --\f
 
 fields

diff --git a/src/Graphic3d/Graphic3d_StructureManager.cxx b/src/Graphic3d/Graphic3d_StructureManager.cxx
index f64b97427358ded9019428ab1f87af5409ff01ac..62324f22fda3a9d1e25607c627138ef1991fb2d1 100644 (file)
--- a/src/Graphic3d/Graphic3d_StructureManager.cxx
+++ b/src/Graphic3d/Graphic3d_StructureManager.cxx
@@ -369,13 +369,26 @@ const Handle(Graphic3d_GraphicDriver)& Graphic3d_StructureManager::GraphicDriver
 
 }
 
-void Graphic3d_StructureManager::ReComputeStructures()
+void Graphic3d_StructureManager::RecomputeStructures()
 {
+  // Go through all unique structures including child (connected) ones and ensure that they are computed.
+  Graphic3d_MapOfStructure aStructNetwork;
+
   for (Graphic3d_MapIteratorOfMapOfStructure anIter(MyDisplayedStructure); anIter.More(); anIter.Next())
   {
     Handle(Graphic3d_Structure) aStructure = anIter.Key();
+    anIter.Key()->Network (anIter.Key(), Graphic3d_TOC_DESCENDANT, aStructNetwork);
+  }
+
+  RecomputeStructures (aStructNetwork);
+}
 
-    aStructure->Clear();
-    aStructure->Compute();
+void Graphic3d_StructureManager::RecomputeStructures (const Graphic3d_MapOfStructure& theStructures)
+{
+  for (Graphic3d_MapIteratorOfMapOfStructure anIter (theStructures); anIter.More(); anIter.Next())
+  {
+    Handle(Graphic3d_Structure) aStruct = anIter.Key();
+    aStruct->Clear();
+    aStruct->Compute();
   }
 }

diff --git a/src/Visual3d/Visual3d_View.cxx b/src/Visual3d/Visual3d_View.cxx
index 6bd3d52e68f1aef7b6cdd3d4b65c9bc3f856e9bf..d5f3db9217666d9a0a09377fb36166a4e74dd9e5 100644 (file)
--- a/src/Visual3d/Visual3d_View.cxx
+++ b/src/Visual3d/Visual3d_View.cxx
@@ -1117,7 +1117,7 @@ void Visual3d_View::Update (const Handle(Visual3d_Layer)& AnUnderLayer, const Ha
 
         if (MyGraphicDriver->IsDeviceLost())
         {
-          MyViewManager->ReComputeStructures();
+          MyViewManager->RecomputeStructures();
           MyGraphicDriver->ResetDeviceLostFlag();
         }
 

diff --git a/tests/bugs/vis/bug24358 b/tests/bugs/vis/bug24358
new file mode 100644 (file)
index 0000000..e6fb3cc
--- /dev/null
+++ b/tests/bugs/vis/bug24358
@@ -0,0 +1,30 @@
+puts "==========="
+puts "OCC24358"
+puts "==========="
+puts ""
+###########################################################################
+# 0024358: TKV3d - connected structures are not re-computed on device lost
+###########################################################################
+
+set Image1 $imagedir/${casename}_1.png
+set Image2 $imagedir/${casename}_2.png
+
+pload MODELING
+pload VISUALIZATION
+box b1 0 0 0 1 2 3
+box b2 3 0 0 3 2 1
+vinit View1
+vclear
+vaxo
+vsetdispmode 1
+vconnectsh b1c -3 0 0 1 0 0 0 0 1 b1 b2
+vfit
+
+vdump $Image1
+
+vclose View1 1
+vinit View1
+vdisplay b1c
+vfit
+
+vdump $Image2