View Issue Details

IDProjectCategoryView StatusLast Update
0032371CommunityOCCT:Modeling Algorithmspublic2022-10-24 10:41
Reporterguseynov.ruslan_163816 Assigned Tomsv  
PrioritynormalSeveritytrivial 
Status newResolutionopen 
Product Version5.2.2 
Target Version7.8.0 
Summary0032371: Modeling Algorithms - Unnecessary range of iteration loop in an algorithm
DescriptionIn file src/BRepLib/BRepLib.cxx, line 411:
for (ii = 0 ; ii < 3 ; ii++ ) {
should be changed to
for (ii = 1 ; ii < 3 ; ii++ ) {
Since ii == 0 always results in obtaining a null pointer Curve2dPtr after calling the first line BRep_Tool::CurveOnSurface (this function expects ii >= 1):

void  BRep_Tool::CurveOnSurface(const TopoDS_Edge& E, 
                                Handle(Geom2d_Curve)& C, 
                                Handle(Geom_Surface)& S, 
                                TopLoc_Location& L,
                                Standard_Real& First,
                                Standard_Real& Last,
                                const Standard_Integer Index)
{
  if (Index < 1)
    return;
TagsNo tags attached.
Test case number

Activities

There are no notes attached to this issue.

Issue History

Date Modified Username Field Change
2021-05-14 12:50 guseynov.ruslan_163816 New Issue
2021-05-14 12:50 guseynov.ruslan_163816 Assigned To => msv
2021-05-14 22:24 kgv Target Version => 7.6.0
2021-05-14 22:24 kgv Summary Unnecessary range of iteration loop in an algorithm => Modeling Algorithms - Unnecessary range of iteration loop in an algorithm
2021-05-14 22:24 kgv Description Updated
2021-05-14 22:27 kgv Product Version 7.5.1 => 7.2.0
2021-05-14 22:32 kgv Product Version 7.2.0 => 5.2.2
2021-08-29 19:24 msv Target Version 7.6.0 => 7.7.0
2022-10-24 10:41 szy Target Version 7.7.0 => 7.8.0