DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0030593

Modeling Algorithms - ShapeUpgrade_ShapeDivide::Perform() throws Standard_NoMoreObject

Open CASCADEOCCT:Modeling Algorithmsclosed11 public notes

Search issues

Description

ShapeUpgrade_ShapeDivide::Perform() throws Standard_NoMoreObject when exceptions are enabled.

Steps to reproduce

Test case bugs/modalg_2/bug21

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Mar 19 11:24:21 2019 +0300

    0030593: Modeling Algorithms - ShapeUpgrade_ShapeDivide::Perform() throws Standard_NoMoreObject
    
    Added missing TopExp_Explorer::More() check.
02Commenter 2
Branch [archived branch] has been updated by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Mar 19 11:31:06 2019 +0300

    BRepCheck_Analyzer::Perform() - add missing TopExp_Explorer::More() check.

03Commenter 2
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Mar 19 11:24:21 2019 +0300

    0030593: Modeling Algorithms - ShapeUpgrade_ShapeDivide::Perform() throws Standard_NoMoreObject
    
    Added missing TopExp_Explorer::More() checks to ShapeUpgrade_ShapeDivide::Perform() and BRepCheck_Analyzer::Perform().
04Author
05Commenter 5
src/BRepCheck/BRepCheck_Analyzer.cxx
- In line 150 the variable exp is always not initialized, so SetFailStatus(exp.Current()) may be simply removed.

Please check the difference in products on Linux:
IMAGE sat read_parallel_1 A1: A1.png differs [15895 different pixels]
06Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
07Author
08Commenter 8
Reviewed.
09Commenter 1
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16631.159999999985 / 16593.449999999997 [+0.23%]
Products
Total CPU difference: 9106.930000000042 / 9102.210000000046 [+0.05%]
Windows-64-VC14:
OCCT
Total CPU difference: 17992.734375 / 17984.46875 [+0.05%]
Products
Total CPU difference: 10544.484375 / 10609.28125 [-0.61%]

Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
10Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]
11Commenter 2
Branch [archived branch] has been deleted by Participant.

[revision removed]