Archived issue #0033019

Coding - get rid of unused classes

Open CASCADEOCCT:Codingclosed5 public notes

Search issues

Description

I have found that the following classes are rarely used or unused completely within OCCT:

src/Bnd/Bnd_BoundSortBox2d.hxx
src/Bnd/Bnd_SeqOfBox.hxx
src/Quantity/Quantity_Array2OfColor.hxx
src/Quantity/Quantity_PhysicalQuantity.hxx

It is worth getting rid of them to clear the code base.

Steps to reproduce

N/A

Public activity

5 archived notes

Participants are labeled by their role within this record.

01Commenter 1
Branch [archived branch] has been created by Author.

[revision removed]


Detailed log of new commits:

Author: Author
Date: Tue Jun 14 17:02:28 2022 +0300

    0033019: Coding - get rid of unused classes
02Commenter 2
Please, be careful, Bnd_SeqOfBox is used within OCC Products (OverlapShape_Check.cxx).
03Commenter 3
Dear Kirill,

Could you please take a look at CR33019 branch in OCCT and Products repository.

Jenkins job is:
http://jenkins-test-occt.nnov.opencascade.com/view/CR33019-master-aml/view/COMPARE/
04Commenter 4
Combination -
OCCT branch : [archived branch]
master SHA - [revision removed]
[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: 18769.010000000537 / 18860.170000000642 [-0.48%]
Products
Total CPU difference: 11750.300000000107 / 11722.3800000001 [+0.24%]
Windows-64-VC14:
OCCT
Total CPU difference: 20650.375 / 20670.390625 [-0.10%]
Products
Total CPU difference: 13263.109375 / 13253.046875 [+0.08%]


Image differences :
No differences that require special attention

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

[revision removed]