DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0023145

Suspicious else (2)

CommunityOCCT:Modeling Algorithmsclosed4 public notes

Search issues

Description

src\LocalAnalysis\LocalAnalysis_SurfaceContinuity.cxx , LocalAnalysis_SurfaceContinuity::SurfC1

There are two if/else that have the same behavoir.

if (norm1v >= norm2v )
        myLambda1V= norm2v / norm1v;
else myLambda1V = norm2v / norm1v;
(... and another below)

Public activity

4 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Else statement was incorrect, is has been changed.

Git branch CR23145 is ready to be reviewed.

Dear Oleg,
Please review.
02Commenter 2
Dear JGV,
Please review.
03Commenter 3
No remarks
04Commenter 4
Dear Commenter 3,
Branch CR23145 (and products from GIT master) was compiled on Linux and Windows platforms and tested.

Regressions:
Not detected

Improvements:
Not detected

Testing cases:
Not needed

Related records