Merge pull request #222 from FriendsOfCADability/ParametricCurve_FixWarnings

Fix warnings in ParametricCurve and dependent classes.

Author: dsn27

CADability/netDxf/GTE/BSplineCurve.cs

@@ -62,7 +62,7 @@ public BSplineCurve(BasisFunctionInput input, Vector3[] controls)
                 controls.CopyTo(this.controls, 0);
             }
 
-            this.isConstructed = true;
+            IsConstructed = true;
         }
 
         // Member access.
@@ -93,7 +93,7 @@ public override void Evaluate(double t, int order, out Vector3[] jet)
             int supOrder = SUP_ORDER;
             jet = new Vector3[supOrder];
 
-            if (!this.isConstructed || order >= supOrder)
+            if (!IsConstructed || order >= supOrder)
             {
                 // Return a zero-valued jet for invalid state.
                 return;

CADability/netDxf/GTE/BezierCurve.cs

@@ -103,7 +103,7 @@ public BezierCurve(Vector3[] controls, int degree)
                 }
             }
 
-            this.isConstructed = true;
+            IsConstructed = true;
         }
 
         // Member access.
@@ -134,7 +134,7 @@ public override void Evaluate(double t, int order, out Vector3[] jet)
             const int supOrder = SUP_ORDER;
             jet = new Vector3[supOrder];
 
-            if (!this.isConstructed || order >= SUP_ORDER)
+            if (!IsConstructed || order >= SUP_ORDER)
             {
                 // Return a zero-valued jet for invalid state.
                 return;

CADability/netDxf/GTE/NURBSCurve.cs

@@ -69,7 +69,7 @@ public NURBSCurve(BasisFunctionInput input, Vector3[] controls, double[] weights
                 weights.CopyTo(this.weights, 0);
             }
 
-            this.isConstructed = true;
+            IsConstructed = true;
         }
 
         // Member access.
@@ -143,7 +143,7 @@ public override void Evaluate(double t, int order, out Vector3[] jet)
             const int supOrder = SUP_ORDER;
             jet = new Vector3[supOrder];
 
-            if (!this.isConstructed || order >= supOrder)
+            if (!IsConstructed || order >= supOrder)
             {
                 // Return a zero-valued jet for invalid state.
                 return;

CADability/netDxf/GTE/ParametricCurve.cs

@@ -42,12 +42,11 @@ public abstract class ParametricCurve
         protected const int DEFAULT_MAX_BISECTIONS = 1024;
         protected const int SUP_ORDER = 4;
 
-        protected readonly double[] times;
-        protected readonly double[] segmentLength;
-        protected readonly double[] acumulatedLength;
-        protected int rombergOrder;
-        protected int maxBisections;
-        protected bool isConstructed;
+        private readonly double[] times;
+        private readonly double[] segmentLength;
+        private readonly double[] acumulatedLength;
+        private int rombergOrder;
+        private int maxBisections;
 
         // Abstract base class for a parameterized curve X(t), where t is the
         // parameter in [tmin,tmax] and X is an N-tuple position.  The first
@@ -66,16 +65,13 @@ protected ParametricCurve(int numSegments, double[] times)
             this.acumulatedLength = new double[numSegments];
             this.rombergOrder = DEFAULT_ROMBERG_ORDER;
             this.maxBisections = DEFAULT_MAX_BISECTIONS;
-            this.isConstructed = false;
+            IsConstructed = false;
         }
 
         // To validate construction, create an object as shown:
         // DerivedClassCurve<N, Real> curve(parameters);
         // if (!curve) { <constructor failed, handle accordingly>; }
-        public bool IsConstructed
-        {
-            get { return this.isConstructed; }
-        }
+        public bool IsConstructed { get; protected set; }
 
         // Member access.
         public double TMin