MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0023760Open CASCADE[OCCT] OCCT:Visualizationpublic2013-02-14 10:412017-07-20 10:51
Reportersan 
Assigned Tokgv 
PrioritylowSeverityintegration request 
StatusassignedResolutionopen 
PlatformAOSLOS VersionL
Product Version[OCCT] 6.5.4 
Target Version[OCCT] 7.4.0*Fixed in Version 
Summary0023760: Visualization - drop redundant class AIS_Triangulation
Description- AIS_Triangulation::Compute(): hasVColors and myFlagColors variables contain identical values, so one of them should be removed.

- AIS_Triangulation::Compute(): the purpose of the following line is unclear:
const TColStd_Array1OfInteger& colors = myColor->Array1();
Is it really needed? Most likely it is to be removed, and myColor variable should be used directly in the loop below.

- General remark concerning AIS_Triangulation class: myFlagColors, myNbNodes and myNbTriangles class fields should be removed as completely useless. Consistency of the color array should be checked in Compute() method and include standard non-null handle check and array size check.

- OCCT coding rules to be applied.
TagsNo tags attached.
Test case number
Attached Files

- Relationships

-  Notes
(0027928)
kgv (developer)
2014-02-17 16:18

This class seems to be not much usable in current state.
Probably better drop it entirely.

Remark by Roman Lygin:
> Why not use MeshVS instead?

- Issue History
Date Modified Username Field Change
2013-02-14 10:41 san New Issue
2013-02-14 10:41 san Assigned To => san
2013-02-14 10:41 san Relationship added related to 0023743
2014-02-17 16:18 kgv Note Added: 0027928
2016-06-16 20:54 kgv Assigned To san => kgv
2016-06-16 20:54 kgv Severity tweak => integration request
2016-06-16 20:54 kgv Status new => assigned
2016-06-16 20:54 kgv Target Version => 7.1.0
2016-06-16 20:54 kgv Summary Revision of AIS_Triangulation class implementation => Visualization - drop redundant class AIS_Triangulation
2016-10-25 09:38 kgv Target Version 7.1.0 => 7.2.0
2017-07-20 10:51 kgv Target Version 7.2.0 => 7.4.0*


Copyright © 2000 - 2018 MantisBT Team
Powered by Mantis Bugtracker