From: azv <azv@opencascade.com>
Date: Wed, 22 Jun 2016 10:10:47 +0000 (+0300)
Subject: PlaneGCSSolver: It is necessary to calculate parameters of fixed arc to avoid conflic... 
X-Git-Tag: V_2.4.0~70
X-Git-Url: http://git.salome-platform.org/gitweb/?a=commitdiff_plain;h=34f77b89b02b81c51ebce0e5a1dd08aa16dff412;p=modules%2Fshaper.git

PlaneGCSSolver: It is necessary to calculate parameters of fixed arc to avoid conflicts (issue #1584)
---

diff --git a/src/SketchSolver/PlaneGCSSolver/PlaneGCSSolver_Storage.cpp b/src/SketchSolver/PlaneGCSSolver/PlaneGCSSolver_Storage.cpp
index 66c47d162..08bddb275 100644
--- a/src/SketchSolver/PlaneGCSSolver/PlaneGCSSolver_Storage.cpp
+++ b/src/SketchSolver/PlaneGCSSolver/PlaneGCSSolver_Storage.cpp
@@ -383,10 +383,6 @@ void PlaneGCSSolver_Storage::verifyFixed()
 
 void PlaneGCSSolver_Storage::processArc(const EntityWrapperPtr& theArc)
 {
-  // no need to constraint a fixed arc
-  if (theArc->group() == GID_OUTOFGROUP)
-    return;
-
   // Calculate additional parameters necessary for PlaneGCS
   const std::list<EntityWrapperPtr>& aSubs = theArc->subEntities();
   std::list<EntityWrapperPtr>::const_iterator aSubIt = aSubs.begin();
@@ -430,8 +426,8 @@ void PlaneGCSSolver_Storage::processArc(const EntityWrapperPtr& theArc)
     *aEndAngle += anAngle;
   }
 
-  // do not constraint copied arc
-  if (anArcFeature->isCopy())
+  // no need to constraint a fixed or a copied arc
+  if (theArc->group() == GID_OUTOFGROUP || anArcFeature->isCopy())
     return;
   // No need to add constraints if they are already exist
   std::map<EntityWrapperPtr, std::vector<GCSConstraintPtr> >::const_iterator