MantisBT - Open CASCADE |
| View Issue Details |
|
| ID | Project | Category | View Status | Date Submitted | Last Update |
| 0023760 | Open CASCADE | [OCCT] OCCT:Visualization | public | 2013-02-14 10:41 | 2020-08-28 16:31 |
|
| Reporter | san | |
| Assigned To | kgv | |
| Priority | low | Severity | integration request | |
| Status | assigned | Resolution | open | |
| Platform | A | OS | L | OS Version | L |
| Product Version | [OCCT] 6.5.4 | |
| Target Version | [OCCT] 7.6.0* | Fixed in Version | | |
| Test case number | |
|
| Summary | 0023760: 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. |
| Steps To Reproduce | |
Additional information
and documentation updates | |
| Tags | No tags attached. |
| Relationships | |
| Attached Files | |
|
| 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 |
| 2019-08-13 12:04 | kgv | Target Version | 7.4.0 => 7.5.0 |
| 2020-08-28 16:31 | kgv | Target Version | 7.5.0 => 7.6.0* |