From 07eebb1c522e8f2b282367df503d5398de1c31c7 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;