From 069fc2a596c62425df49499ec18f2cb53123f4c6 Mon Sep 17 00:00:00 2001
From: san <san@opencascade.com>
Date: Fri, 20 Mar 2009 16:30:48 +0000
Subject: [PATCH] Optimizing selection highlighting Invoking ModifiedEvent for
 the selector when its selection mode changes

---
 src/SVTK/SVTK_Functor.h    | 31 +++++++++++++++++++++++++++++++
 src/SVTK/SVTK_Selector.cxx |  2 ++
 src/SVTK/SVTK_View.cxx     | 31 +++++++++++++++++++++++--------
 3 files changed, 56 insertions(+), 8 deletions(-)

diff --git a/src/SVTK/SVTK_Functor.h b/src/SVTK/SVTK_Functor.h
index d76a93548..84548a429 100644
--- a/src/SVTK/SVTK_Functor.h
+++ b/src/SVTK/SVTK_Functor.h
@@ -26,6 +26,9 @@
 #include <VTKViewer_Functor.h>
 
 #include "SALOME_InteractiveObject.hxx"
+#include "SALOME_ListIO.hxx"
+
+#include <TColStd_MapOfTransient.hxx>
 
 /*!
   \file SVTK_Functor.h
@@ -82,6 +85,34 @@ namespace SVTK
     }
   };
 
+  //----------------------------------------------------------------
+  //! This functor check, if the actor's #SALOME_InteractiveObject is in the given list
+  template<class TActor> 
+  struct TIsInList
+  {
+    TColStd_MapOfTransient myIObjects;
+    //! To construct the functor
+    TIsInList( const SALOME_ListIO& theObjList )
+    {
+      SALOME_ListIteratorOfListIO anIter( theObjList );
+      for(; anIter.More(); anIter.Next() ){
+        if ( !myIObjects.Contains( anIter.Value() ) )
+          myIObjects.Add( anIter.Value() );
+      }
+    }
+
+    //! To calculate the functor
+    bool operator()(TActor* theActor)
+    {
+      if(theActor->hasIO())
+      {
+	Handle(SALOME_InteractiveObject) anIO = theActor->getIO();
+        return myIObjects.Contains( anIO );
+      }
+      return false;
+    }
+  };
+
 
   //----------------------------------------------------------------
   /*!
diff --git a/src/SVTK/SVTK_Selector.cxx b/src/SVTK/SVTK_Selector.cxx
index 906574ca7..1945fbc17 100644
--- a/src/SVTK/SVTK_Selector.cxx
+++ b/src/SVTK/SVTK_Selector.cxx
@@ -125,6 +125,8 @@ SVTK_SelectorDef
     mySelectionMode = theMode;
     myMapIOSubIndex.clear();
     this->EndPickCallback();
+    // Let everybody process selection mode change
+    this->Modified();
   }
 }
 
diff --git a/src/SVTK/SVTK_View.cxx b/src/SVTK/SVTK_View.cxx
index 57e6e49eb..676df5cc2 100644
--- a/src/SVTK/SVTK_View.cxx
+++ b/src/SVTK/SVTK_View.cxx
@@ -40,6 +40,8 @@
 #include <vtkActorCollection.h>
 #include <vtkRenderer.h>
 
+#include <set>
+
 /*!
   Constructor
 */
@@ -134,6 +136,22 @@ namespace SVTK
       }
     }
   };
+
+  struct TAddAction
+  {
+    std::set<SALOME_Actor*>* myActors;
+    TAddAction( std::set<SALOME_Actor*>* actors ):
+      myActors( actors )
+    {}
+    
+    void
+    operator()( SALOME_Actor* theActor) 
+    {
+      if(myActors && theActor->GetMapper() && theActor->hasIO()){
+	myActors->insert( theActor );
+      }
+    }    
+  };
 }
 
 /*!
@@ -146,16 +164,13 @@ SVTK_SignalHandler
   vtkActorCollection* anActors = myMainWindow->getRenderer()->GetActors();
 
   using namespace SVTK;
-  ForEach<SALOME_Actor>(anActors,
-			THighlightAction( false ));
   SVTK_Selector* aSelector = myMainWindow->GetSelector();
   const SALOME_ListIO& aListIO = aSelector->StoredIObjects();
-  SALOME_ListIteratorOfListIO anIter(aListIO);
-  for(; anIter.More(); anIter.Next()){
-    ForEachIf<SALOME_Actor>(anActors,
-			    TIsSameIObject<SALOME_Actor>(anIter.Value()),
-			    THighlightAction(true));
-  }
+
+  ForEachIfElse<SALOME_Actor>(anActors,
+                              TIsInList<SALOME_Actor>(aListIO),
+                              THighlightAction(true),
+                              THighlightAction(false));
 
   myMainWindow->Repaint(false);
 }
-- 
2.39.2