Archived issue #0022564

BRepMesh_Classifier improvements

Open CASCADEOCCT:Meshclosed11 public notes

Search issues

Description

After some analysis of BRepMesh_Classifier code is proposed to implement the
next code changes:
* remove copy/paste from constructor etc
* remove check on natural restriction from this class, and usage of this field
from BRepMesh_FastDiscreteFace

Public activity

11 archived notes

Participants are labeled by their role within this record.

01Commenter 1
*** Bug 22563 has been marked as a duplicate of this bug. ***
02Commenter 2
Oleg, please review the last fix
03Commenter 2
Fix source code passed review without remarks.
04Commenter 4
This fix does not require a special test script
05Commenter 5
This bug has been tested( without regression ) and reviewed. Please integrate this fix in ros
06Commenter 6
Dear Ekaterina,
please create the branche for this fix.
07Commenter 7
Dear Mikhail,

According to current policy, please create branch by yourself for the new bugs that have been created and fixed before branching time. Ask me if you have any problem or question concerning the merging with trunk
08Commenter 8
SVN branche http://svn/svn/occt/branches/22564 created from Attachment 3 (GZ)
Dear Oleg,
Could you please revise fix http://svn/svn/occt/branches/22564
09Commenter 2
Dear Commenter 1,
branch has been revised, merged, updated and ready for long-expected integration.
10Commenter 10
Dear Commenter 1,
Workbench KAS:dev:mkv-OCC22564-occt has been created from SVN branch http://svn/svn/occt/branches/22564
(and mkv-OCC22564-products from trank) and compiled on Linux platform.

There are not regressions in mkv-OCC22564-products regarding to KAS:dev:products-20110821-opt

See results in /dn45/occttests/results/KAS/dev/mkv-OCC22564-products_01092011/lin
See reference results in /dn45/occttests/results/KAS/dev/products-20110821-opt_22082011/lin
See test cases in /dn45/occttests/tests/ED
N.B. In order to launch testing case you can make use the following instructions
http://doc/doku.php?id=occt:certification
11Commenter 1
Integration into occt repository

Date: 2011-09-02 12:16:09 +0400 (Fri, 02 Sep 2011)
New Revision: 8940

Modified:
   trunk/src/BRepMesh/BRepMesh_Classifier.cdl
   trunk/src/BRepMesh/BRepMesh_Classifier.cxx
   trunk/src/BRepMesh/BRepMesh_Classifier.lxx
   trunk/src/BRepMesh/BRepMesh_Delaun.cdl
   trunk/src/BRepMesh/BRepMesh_Delaun.cxx
   trunk/src/BRepMesh/BRepMesh_FastDiscret.cdl
   trunk/src/BRepMesh/BRepMesh_FastDiscret.cxx
   trunk/src/BRepMesh/BRepMesh_FastDiscretFace.cxx
   trunk/src/BRepMesh/BRepMesh_IncrementalMesh.cxx

Related records