From 34f77b89b02b81c51ebce0e5a1dd08aa16dff412 Mon Sep 17 00:00:00 2001
From: azv <azv@opencascade.com>
Date: Wed, 22 Jun 2016 13:10:47 +0300
Subject: [PATCH] PlaneGCSSolver: It is necessary to calculate parameters of
 fixed arc to avoid conflicts (issue #1584)

---
 .../PlaneGCSSolver/PlaneGCSSolver_Storage.cpp             | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

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
-- 
2.39.2