Archived issue #0031120

Memory leak in BRepOffsetAPI_MakeOffset

Open CASCADEOCCT:Modeling Algorithmsclosed16 public notes

Description

There is memory leak when offsetting a wire using BRepOffsetAPI_MakeOffset.

Steps to reproduce

restore wire.brep w

for {set i 0} {$i < 1000} {incr i} {
  mkoffset r w 1 0.5
  if ![expr $i%100] {
    puts "i = $i, mem = [lindex [meminfo h] 0]"
  }
}

Output:
i = 0, mem = 8723622
i = 100, mem = 9464422
i = 200, mem = 10205222
i = 300, mem = 10946022
i = 400, mem = 11686822
i = 500, mem = 12427622
i = 600, mem = 13168422
i = 700, mem = 13909222
i = 800, mem = 14650022
i = 900, mem = 15390822

Public activity

16 archived notes

Participants are labeled by their role within this record.

01Commenter 12019-11-01 15:30

Branch [archived branch] has been created by Author.

[revision removed]

Detailed log of new commits:

Author: Author
Date: Fri Nov 1 15:05:43 2019 +0300

0031120: Memory leak in BRepOffsetAPI_MakeOffset

02Commenter 22019-11-01 15:31

This is a beginning patch.

03Commenter 12019-11-15 14:16

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

04Commenter 42019-11-15 14:19

Please add the attached shape to database.

05Commenter 52019-11-18 09:56

Done.

06Commenter 12019-11-19 18:17

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

07Commenter 12019-11-20 17:12

Branch [archived branch] has been updated by Participant.

[revision removed]

Detailed log of new commits:

Author: jgv
Date: Wed Nov 20 17:10:32 2019 +0300

Fix of regression

08Commenter 82019-11-20 17:47

Please review the branch CR31120.

09Commenter 92019-11-20 18:09

The following changes are useless:
src/MAT/MAT_Bisector.cxx
src/MAT/MAT_Bisector.hxx
src/MAT/MAT_Edge.cxx
src/MAT/MAT_Edge.hxx

The test case needs to be changed so as to generate error in case of memory leak.

10Commenter 12019-12-16 20:30

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

11Commenter 12019-12-17 15:02

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

12Commenter 12019-12-17 15:54

Branch [archived branch] has been updated forcibly by Participant.

[revision removed]

13Commenter 132019-12-18 13:56

Please review the branch CR31120.

14Commenter 142019-12-18 14:35

Reviewed.

15Commenter 22019-12-19 16:35

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: 16963.310000000132 / 16976.290000000114 [-0.08%]
Products
Total CPU difference: 10729.300000000072 / 10746.800000000061 [-0.16%]
Windows-64-VC14:
OCCT
Total CPU difference: 18395.1875 / 18406.625 [-0.06%]
Products
Total CPU difference: 12706.875 / 12757.328125 [-0.40%]

Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention

16Commenter 12019-12-22 11:10

Branch [archived branch] has been deleted by Participant.

[revision removed]