From: bourcier <bourcier>
Date: Wed, 6 Nov 2013 09:41:55 +0000 (+0000)
Subject: Remove NotifySubMeshesHypothesisModification for performance reason for method called... 
X-Git-Tag: V7_3_0a1~8
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=0b270f3ff3a5779d75f952a7e14ceb77be929cce;p=plugins%2Fblsurfplugin.git

Remove NotifySubMeshesHypothesisModification for performance reason for method called only via python.
Multiple (thousands) calls to NotifySubMeshesHypothesisModification slow down the whole meshing process.
---

diff --git a/src/BLSURFPlugin/BLSURFPlugin_Hypothesis.cxx b/src/BLSURFPlugin/BLSURFPlugin_Hypothesis.cxx
index 444429e..51f0d22 100644
--- a/src/BLSURFPlugin/BLSURFPlugin_Hypothesis.cxx
+++ b/src/BLSURFPlugin/BLSURFPlugin_Hypothesis.cxx
@@ -1289,7 +1289,9 @@ void BLSURFPlugin_Hypothesis::AddFacePeriodicity(TEntry theFace1Entry, TEntry th
 
   _facesPeriodicityVector.push_back(pairOfFacesEntries);
 
-  NotifySubMeshesHypothesisModification();
+  // Removed for performance reason since AddFacePeriodicity is called multiple times (one time for each face)
+  // Does not affect the behaviour since it is only called via python, not via GUI.
+  //NotifySubMeshesHypothesisModification();
 }
 
 
@@ -1307,7 +1309,9 @@ void BLSURFPlugin_Hypothesis::AddEdgePeriodicity(TEntry theFace1Entry, TEntry th
 
   _edgesPeriodicityVector.push_back(edgePeriodicity);
 
-  NotifySubMeshesHypothesisModification();
+  // Removed for performance reason since AddEdgePeriodicity is called multiple times (one time for each edge)
+  // Does not affect the behaviour since it is only called via python, not via GUI.
+  //NotifySubMeshesHypothesisModification();
 }
 
 //=======================================================================
@@ -1323,7 +1327,9 @@ void BLSURFPlugin_Hypothesis::AddVertexPeriodicity(TEntry theEdge1Entry, TEntry
 
   _verticesPeriodicityVector.push_back(vertexPeriodicity);
 
-  NotifySubMeshesHypothesisModification();
+  // Removed for performance reason since AddVertexPeriodicity is called multiple times (one time for each vertex)
+  // Does not affect the behaviour since it is only called via python, not via GUI.
+  //NotifySubMeshesHypothesisModification();
 }
 
 //=============================================================================