Archived issue #0030366

Coding Rules - GeometryTest_API2dCommands.cxx seems to be excess

Open CASCADEOCCT:Codingclosed5 public notes

Search issues

Description

Currently the file GeometryTest_API2dCommands.cxx defines some DRAW-commands. However, names of these commands are duplicate of analogical commands in the file GeomliteTest_API2dCommands.cxx, which are actively used by current OCCT-version.

So, is the file GeometryTest_API2dCommands.cxx really usefull?

P.S.

Please note, that calling of the method GeometryTest::API2dCommands(...) is commented in the file GeometryTest.cxx.

Public activity

5 archived notes

Participants are labeled by their role within this record.

01Commenter 3
This change was introduced in '1997.
> jpi 09/06/97 utilisation des commandes de GeomliteTest

02Commenter 4
Branch [archived branch] has been created by Commenter 3.

[revision removed]


Detailed log of new commits:

Author: Commenter 3
Date: Fri Feb 1 17:49:08 2019 +0300

    0030366: Coding Rules - GeometryTest_API2dCommands.cxx seems to be excess
03Commenter 3
Please take trivial patch.
04Commenter 1
tested in framework WEEK-6 branch


Combination -
OCCT branch : [archived branch] SHA - [revision removed]
Products branch : [archived branch] SHA - [revision removed]
was compiled on Linux, MacOS and Windows platforms and tested in optimize mode.

Number of compiler warnings:
No new/fixed warnings

Regressions/Differences/Improvements:
No regressions/differences

CPU differences:
Debian80-64:
OCCT
Total CPU difference: 16368.269999999928 / 16363.790000000015 [+0.03%]
Products
Total CPU difference: 7176.410000000036 / 7177.480000000016 [-0.01%]
Windows-64-VC14:
OCCT
Total CPU difference: 17772.359375 / 17773.921875 [-0.01%]
Products
Total CPU difference: 8601.046875 / 8604.828125 [-0.04%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
05Commenter 4
Branch [archived branch] has been deleted by Commenter 3.

[revision removed]

Related records