MantisBT - Community
View Issue Details
0022938Community[OCCT] OCCT:Modeling Algorithmspublic2012-01-30 15:022012-04-16 12:19
Pawel 
bugmaster 
normalminor 
closedfixed 
[OCCT] 6.5.2 
[OCCT] 6.5.3[OCCT] 6.5.3 
Test case is not required
0022938: HLRBRep_InternalAlgo.cxx
the following variables in HLRBRep_InternalAlgo::HideSelected (lines 800-820 in OCCT 6.5.2):

Standard_Integer *Val = new Standard_Integer [nf+1];
Standard_Real *Size = new Standard_Real [nf+1];
Standard_Integer *Index = new Standard_Integer [nf+1];

are deleted (lines 918-920) the following way:

delete Val;
delete Size;
delete Index;

which I guess is wrong because the array delete operator should be used like in OCCT 6.5.1:

delete [] Val;
delete [] Size;
delete [] Index;
No tags attached.
Issue History
2012-01-30 15:02PawelNew Issue
2012-01-30 15:02PawelAssigned To => mkv
2012-01-30 15:04PawelNote Added: 0019308
2012-01-30 16:24bugmasterCategoryOCCT:DRAW => OCCT:Modeling Algorithms
2012-01-30 16:24bugmasterProduct Version5.0.0 => 6.5.2
2012-01-30 16:24bugmasterAssigned Tomkv => jgv
2012-01-30 16:24bugmasterStatusnew => assigned
2012-01-30 17:38abvAssigned Tojgv => dbv
2012-01-30 17:39abvNote Added: 0019312
2012-02-07 11:49dbvNote Added: 0019426
2012-02-07 11:49dbvAssigned Todbv => abv
2012-02-07 11:49dbvStatusassigned => resolved
2012-02-07 12:07abvNote Added: 0019430
2012-02-07 12:07abvStatusresolved => reviewed
2012-02-07 12:07abvTarget Version => 6.5.3
2012-02-14 13:12mkvAssigned Toabv => aan
2012-02-15 16:06apnNote Added: 0019581
2012-02-15 16:06apnNote Edited: 0019581bug_revision_view_page.php?bugnote_id=19581#r3466
2012-02-15 16:07apnTest case number => Test case is not required
2012-02-15 16:07apnStatusreviewed => tested
2012-02-17 15:12bugmasterNote Added: 0019625
2012-02-17 15:12bugmasterStatustested => verified
2012-02-17 15:12bugmasterResolutionopen => fixed
2012-02-17 15:12bugmasterAssigned Toaan => Pawel
2012-03-29 17:26bugmasterChangeset attached => occt master b8f2022a
2012-04-16 12:18PawelAssigned ToPawel => bugmaster
2012-04-16 12:19PawelNote Added: 0020390

Notes
(0019308)
Pawel   
2012-01-30 15:04   
Because of the mantis bug reported here (http://dev.opencascade.org/index.php?q=node/39 [^]) I was making some experiments with the input parameters and so some of them are wrong in this report. The correct ones are:

Product Version: [OCCT] 6.5.2
Category: [OCCT] OCCT: Modeling algorithms
(0019312)
abv   
2012-01-30 17:39   
Please use array classes from TColStd package instead of C pointers
(0019426)
dbv   
2012-02-07 11:49   
C pointers has been replaced by array classes from TColStd package

Branch http://svn/svn/occt/branches/OCC22938 [^] is ready to be reviewed.

Dear Andrey,
Please review.
(0019430)
abv   
2012-02-07 12:07   
No remarks, please test
(0019581)
apn   
2012-02-15 16:06   
Dear BugMaster,
  Workbench KAS:dev:apn-22938-occt was created from SVN branch http://svn/svn/occt/branches/OCC22938 [^]
  (and apn-22938-products from trunk) and compiled on Linux and Windows platforms.
  
  There are not regressions in apn-22938-products regarding to KAS:dev:products-20120127-opt
  
  See results in /QADisk/occttests/results/KAS/dev/ apn-22938-products_14022012/lin
  See reference results in /QADisk/occttests/results/KAS/dev/products-20120127-opt_28012012/lin
  See test cases in /QADisk/occttests/tests/ED

(0019625)
bugmaster   
2012-02-17 15:12   
Integrated into trunk of occt repository

Date: 2012-02-17 15:11:03 +0400 (Fri, 17 Feb 2012)
New Revision: 10502

Modified:
   trunk/src/HLRBRep/HLRBRep_InternalAlgo.cxx
(0020390)
Pawel   
2012-04-16 12:19   
Verified. Please close.