Update workaround which prevents an infinite loop in aCutes GetStarPoints method.

See #58

Author: wo80

src/Triangle/NewLocation.cs

@@ -22,10 +22,6 @@ class NewLocation
     {
         const double EPS = 1e-50;
 
-        // A safeguard to prevent an infinite loop in GetStarPoints method, see
-        // https://github.com/wo80/Triangle.NET/issues/58
-        const int GETSTARPOINTS_SAFEGUARD = 2000;
-
         IPredicates predicates;
 
         Mesh mesh;
@@ -2295,7 +2291,11 @@ private int GetStarPoints(Otri badotri, Vertex p, Vertex q, Vertex r,
 
                     if (numvertices == points.Length)
                     {
-                        Array.Resize(ref points, numvertices * 2);
+                        // See https://github.com/wo80/Triangle.NET/issues/58
+                        Log.Instance.Warning("aCute smoothing ran out of precision", "NewLocation.GetStarPoints()");
+
+                        numvertices = 0;
+                        break;
                     }
 
                     // add a new point to the list of surrounding points
@@ -2310,12 +2310,6 @@ private int GetStarPoints(Otri badotri, Vertex p, Vertex q, Vertex r,
                     break;
                 }
 
-                if (numvertices > GETSTARPOINTS_SAFEGUARD)
-                {
-                    numvertices = 0;
-                    break;
-                }
-
             } while (!((Math.Abs(returnPoint[0] - third_x) <= EPS) &&
                      (Math.Abs(returnPoint[1] - third_y) <= EPS)));
             return numvertices / 2;