Fixed warnings that appear when building with MSVC (#1004)

- Refactored loop control flow to use an `if` statement instead of a `for` loop with immediate `break`
- Changed variable type from `int` to `size_t` to match the expected type and eliminate conversion warnings
This commit is contained in:
Dmitrii Kulikov
2026-01-14 14:49:43 +00:00
committed by GitHub
parent a36bca57a9
commit c639199c01
2 changed files with 3 additions and 3 deletions

View File

@@ -202,7 +202,7 @@ IntegResult GaussComposite(Function& theFunc,
const double aH = (theUpper - theLower) / theNbIntervals;
double aSum = 0.0;
int aTotalPoints = 0;
size_t aTotalPoints = 0;
for (int i = 0; i < theNbIntervals; ++i)
{

View File

@@ -277,7 +277,8 @@ TEST_F(BOPAlgo_PaveFillerTest, FuseConeWithRemovedPCurve_NullPCurveHandling)
TopoDS_Wire aNewWire;
aBuilder.MakeWire(aNewWire);
for (TopExp_Explorer aWireExp(aConicalFace, TopAbs_WIRE); aWireExp.More(); aWireExp.Next())
TopExp_Explorer aWireExp(aConicalFace, TopAbs_WIRE);
if (aWireExp.More()) // Only process first wire
{
const TopoDS_Wire& aWire = TopoDS::Wire(aWireExp.Current());
for (TopExp_Explorer anEdgeExp(aWire, TopAbs_EDGE); anEdgeExp.More(); anEdgeExp.Next())
@@ -292,7 +293,6 @@ TEST_F(BOPAlgo_PaveFillerTest, FuseConeWithRemovedPCurve_NullPCurveHandling)
aBuilder.Add(aNewWire, anEdge);
}
}
break; // Only process first wire
}
// Create new face with the modified wire