OCCT3D
DiscussionsIssue archiveOCCT:Modeling Algorithms

Archived issue #0022937

Wrong delete operator in GeomFill_EvolvedSection.cxx / GeomFill_UniformSection.cxx

CommunityOCCT:Modeling Algorithmsclosed7 public notes

Search issues

Description

GeomFill_EvolvedSection.cxx (line 53) / GeomFill_UniformSection.cxx (line 52) should use the "delete []" operator as arrays are deleted

Public activity

7 archived notes

Participants are labeled by their role within this record.

01Commenter 1
As both these places use buffer just to make some name for variable to be dumped for debug, please replace this by using static buffer (e.g. char buf[256] -- should be pretty sufficient for this in all cases).
02Commenter 2
Dynamic variable allocation replaced by static

Branch http://svn/svn/occt/branches/OCC22937 is ready to be reviewed.

Dear Andrey,
Please review.
03Commenter 3
No remarks, please test
04Commenter 4
Dear Commenter 1,
Workbench KAS:dev:mkv-22937-occt was created from SVN branch http://svn/svn/occt/branches/OCC22937
(and mkv-22937-products from trunk) and compiled on Linux platform.

05Commenter 2
There not regressions in mkv-22937-products regarding to KAS:dev:products-20120210-opt
 
 See results in /QADisk/occttests/results/KAS/dev/mkv-22937-products_10022012/lin
 See reference results in /QADisk/occttests/results/KAS/dev/products-20120210-opt_10022012/lin
 See test cases in /QADisk/occttests/tests/ED
06Commenter 1
Integrated into trunk of occt repository

Date: 2012-02-17 16:08:26 +0400 (Fri, 17 Feb 2012)
New Revision: 10509

Modified:
   trunk/src/GeomFill/GeomFill_EvolvedSection.cxx
   trunk/src/GeomFill/GeomFill_UniformSection.cxx
07Author
Verified. Please close.