0030754: Coding - the array of weights should begin with Lower, not the constant... IR-2019-06-14 master
authorkgv <kgv@opencascade.com>
Wed, 5 Jun 2019 06:34:21 +0000 (09:34 +0300)
committerbugmaster <bugmaster@opencascade.com>
Fri, 14 Jun 2019 15:58:49 +0000 (18:58 +0300)
src/Geom/Geom_BSplineCurve.cxx
src/Geom2d/Geom2d_BSplineCurve.cxx

index 9fc6eef..87e6c9f 100644 (file)
@@ -81,20 +81,17 @@ static void CheckCurveData
     throw Standard_ConstructionError("BSpline curve: # Poles and degree mismatch");
 }
 
-//=======================================================================
-//function : Rational
-//purpose  : check rationality of an array of weights
-//=======================================================================
-
-static Standard_Boolean Rational(const TColStd_Array1OfReal& W)
+//! Check rationality of an array of weights
+static Standard_Boolean Rational (const TColStd_Array1OfReal& theWeights)
 {
-  Standard_Integer i, n = W.Length();
-  Standard_Boolean rat = Standard_False;
-  for (i = 1; i < n; i++) {
-    rat =  Abs(W(i) - W(i+1)) > gp::Resolution();
-    if (rat) break;
+  for (Standard_Integer i = theWeights.Lower(); i < theWeights.Upper(); i++)
+  {
+    if (Abs (theWeights[i] - theWeights[i + 1]) > gp::Resolution())
+    {
+      return Standard_True;
+    }
   }
-  return rat;
+  return Standard_False;
 }
 
 //=======================================================================
index 829e9be..1cca78c 100644 (file)
@@ -79,20 +79,17 @@ static void CheckCurveData
     throw Standard_ConstructionError("BSpline curve: # Poles and degree mismatch");
 }
 
-//=======================================================================
-//function : Rational
-//purpose  : check rationality of an array of weights
-//=======================================================================
-
-static Standard_Boolean Rational(const TColStd_Array1OfReal& W)
+//! Check rationality of an array of weights
+static Standard_Boolean Rational (const TColStd_Array1OfReal& theWeights)
 {
-  Standard_Integer i, n = W.Length();
-  Standard_Boolean rat = Standard_False;
-  for (i = 1; i < n; i++) {
-    rat =  Abs(W(i) - W(i+1)) > gp::Resolution();
-    if (rat) break;
+  for (Standard_Integer i = theWeights.Lower(); i < theWeights.Upper(); i++)
+  {
+    if (Abs (theWeights[i] - theWeights[i + 1]) > gp::Resolution())
+    {
+      return Standard_True;
+    }
   }
-  return rat;
+  return Standard_False;
 }
 
 //=======================================================================