From 3d361b1caab63df5f4cffe4e8524a8e18c63f6a5 Mon Sep 17 00:00:00 2001
From: Abdullah Tahiri <abdullah.tahiri.yo@gmail.com>
Date: Sat, 19 Dec 2020 08:47:43 +0100
Subject: [PATCH] Sketcher: Sketch.cpp clean up old interface

---
 src/Mod/Sketcher/App/Sketch.cpp | 110 --------------------------------
 src/Mod/Sketcher/App/Sketch.h   |   3 -
 2 files changed, 113 deletions(-)

diff --git a/src/Mod/Sketcher/App/Sketch.cpp b/src/Mod/Sketcher/App/Sketch.cpp
index 6ff57bde8b..361dbd5aa3 100644
--- a/src/Mod/Sketcher/App/Sketch.cpp
+++ b/src/Mod/Sketcher/App/Sketch.cpp
@@ -3812,116 +3812,6 @@ Base::Vector3d Sketch::getPoint(int geoId, PointPos pos) const
     return Base::Vector3d();
 }
 
-bool Sketch::hasDependentParameters(int geoId, PointPos pos) const
-{
-    try {
-        geoId = checkGeoId(geoId);
-    }
-    catch (Base::Exception&) {
-        return false;
-    }
-
-    if(Geoms[geoId].external)
-        return true;
-
-    switch(Geoms[geoId].type) {
-        case Point:
-        {
-            switch(pos) { // NOTE: points are added to all the cases, see addition.
-                case none: return Points[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case Line:
-        {
-            switch(pos) {
-                case none: return Lines[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return false;break;
-            }
-        }
-        break;
-        case Arc:
-        {
-            switch(pos) {
-                case none: return Arcs[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case Circle:
-        {
-            switch(pos) { // NOTE: points are added to all the cases, see addition.
-                case none: return Circles[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return false;break;
-                case end: return false;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case Ellipse:
-        {
-            switch(pos) { // NOTE: points are added to all the cases, see addition.
-                case none: return Ellipses[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return false;break;
-                case end: return false;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case ArcOfEllipse:
-        {
-            switch(pos) {
-                case none: return ArcsOfEllipse[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case ArcOfHyperbola:
-        {
-            switch(pos) {
-                case none: return ArcsOfHyperbola[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case ArcOfParabola:
-        {
-            switch(pos) {
-                case none: return ArcsOfParabola[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return Points[Geoms[geoId].midPointId].hasDependentParameters;break;
-            }
-        }
-        break;
-        case BSpline:
-        {
-            switch(pos) {
-                case none: return BSplines[Geoms[geoId].index].hasDependentParameters;break;
-                case start: return Points[Geoms[geoId].startPointId].hasDependentParameters;break;
-                case end: return Points[Geoms[geoId].endPointId].hasDependentParameters;break;
-                case mid: return false;break;
-            }
-        }
-        break;
-        case None:
-            return false; break;
-    }
-
-    return false;
-}
-
 TopoShape Sketch::toShape(void) const
 {
     TopoShape result;
diff --git a/src/Mod/Sketcher/App/Sketch.h b/src/Mod/Sketcher/App/Sketch.h
index c1beac27b8..8448935830 100644
--- a/src/Mod/Sketcher/App/Sketch.h
+++ b/src/Mod/Sketcher/App/Sketch.h
@@ -98,9 +98,6 @@ public:
     /// retrieves a point
     Base::Vector3d getPoint(int geoId, PointPos pos) const;
 
-    /// retrieves whether a geometry has dependent parameters or not
-    bool hasDependentParameters(int geoId, PointPos pos) const;
-
     // Inline methods
     inline bool hasConflicts(void) const { return !Conflicting.empty(); }
     inline const std::vector<int> &getConflicting(void) const { return Conflicting; }