Archived issue #0032696

Coding - get rid of unused forward declarations [StepSelect to Xw]

Open CASCADEOCCT:Codingclosed8 public notes

Search issues

Description

include-what-you-use is a great tool for code health improvement. It allows for detecting unused headers and forward declarations.

Packages from StepSelect to Xw (inclusively) are processed in this ticket. Also, lower-case packages (gp and gce) should be handled as well.

Steps to reproduce

N/A

Public activity

8 archived notes

Participants are labeled by their role within this record.

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

[revision removed]


Detailed log of new commits:

Author: Author
Date: Mon Nov 29 11:18:12 2021 +0300

    0032696: Coding - get rid of unused forward declarations [StepSelect to Xw]
    
    Packages starting with lower-case (gp and gce) are also processed.
02Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
03Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
04Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
05Commenter 2
Branch [archived branch] has been updated forcibly by Author.

[revision removed]
06Commenter 6
Dear Kirill,
Please take a look at the patch:

Jenkins compilation results are here:
http://jenkins-test-occt.nnov.opencascade.com/view/CR32696-master-aml/view/ALL/
07Commenter 7
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: 18166.60000000042 / 18390.6500000004 [-1.22%]
Products
Total CPU difference: 11658.820000000103 / 11648.280000000103 [+0.09%]
Windows-64-VC14:
OCCT
Total CPU difference: 19937.84375 / 19928.171875 [+0.05%]
Products
Total CPU difference: 13069.28125 / 13059.03125 [+0.08%]


Image differences :
No differences that require special attention

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

[revision removed]

Related records