From 0f8db07e90c8f1db6c14a3aa3baf9b285397b18d Mon Sep 17 00:00:00 2001
From: Abdullah Tahiri <abdullah.tahiri.yo@gmail.com>
Date: Tue, 10 Jan 2017 23:22:21 +0100
Subject: [PATCH] Sketcher: Constraint Third Redefinition for internal geometry

=============================================================

Internal alignment of BSplines requires an extra index for unbounded same-type internal geometry (poles, probably also in the future knots).

Value shall not be used, as it interferes with the update mechanism for constraint values.

An alternative solution would be to add another int to all the constraints just for this case.

As internal geometry does not use Third, it seems reasonable to reuse it for this indexing purpose.
---
 src/Mod/Sketcher/App/Constraint.h | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/Mod/Sketcher/App/Constraint.h b/src/Mod/Sketcher/App/Constraint.h
index f7b78de9d8..470ee8027a 100644
--- a/src/Mod/Sketcher/App/Constraint.h
+++ b/src/Mod/Sketcher/App/Constraint.h
@@ -110,7 +110,7 @@ public:
     PointPos FirstPos;
     int Second;
     PointPos SecondPos;
-    int Third;
+    int Third; // Note: for InternalAlignment Type this index indexes equal internal geometry elements (e.g. index of pole in a bspline), thirdpos remains unused. It is not a GeoId. 
     PointPos ThirdPos;
     float LabelDistance;
     float LabelPosition;