lili/OCCT
1
0
Fork 0
mirror of https://github.com/Open-Cascade-SAS/OCCT.git synced 2026-05-10 09:30:48 +08:00
Code Issues Packages Projects Releases Wiki Activity

Modeling - Fix 0-based index in BRep_Tool::CurveOnSurface call (#949)

Browse Source 
Fixed incorrect loop index initialization in GlueEdgesWithPCurves function.
The loop for iterating PCurves started from index 0, but BRep_Tool::CurveOnSurface
requires 1-based indexing (returns immediately when Index < 1).

This bug caused the first PCurve to be skipped, potentially leading to
incorrect edge gluing results in ShapeUpgrade_UnifySameDomain.

Changed: for (int aCurveIndex = 0;; ...) -> for (int aCurveIndex = 1;; ...)
This commit is contained in:
Pasukhin Dmitry
2025-12-23 23:55:16 +00:00
committed by GitHub
parent bb0d8200fa
commit b3052a4002
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
src/ModelingAlgorithms/TKShHealing/ShapeUpgrade/ShapeUpgrade_UnifySameDomain.cxx
View File

@@ -1557,7 +1557,7 @@ static TopoDS_Edge GlueEdgesWithPCurves(const TopTools_SequenceOfShape& aChain,
TColGeom_SequenceOfSurface SurfSeq;
NCollection_Sequence<TopLoc_Location> LocSeq;
for (int aCurveIndex = 0;; aCurveIndex++)
for (int aCurveIndex = 1;; aCurveIndex++)
{
Handle(Geom2d_Curve) aCurve;
Handle(Geom_Surface) aSurface;