DiscussionsIssue archiveOCCT:Visualization

Archived issue #0028644

Visualization, AIS_MultipleConnectedInteractive - remove unused private class SelectMgr_AssemblyEntityOwner

Open CASCADEOCCT:Visualizationclosed6 public notes

Search issues

Description

Compiler says:
/occt/src/AIS/AIS_MultipleConnectedInteractive.cxx|75|warning: ‘{anonymous}::SelectMgr_AssemblyEntityOwner::SelectMgr_AssemblyEntityOwner(opencascade::handle<SelectMgr_EntityOwner>, SelectMgr_SelectableObject*)’ defined but not used [-Wunused-function]|

and indeed, this class is never used since #0024623

Steps to reproduce

N/A

Public activity

6 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: Sat Apr 8 15:03:18 2017 +0300

    0028644: Visualization, AIS_MultipleConnectedInteractive - remove unused private class SelectMgr_AssemblyEntityOwner
02Author
Patch is ready for review.
03Commenter 3
Dear Commenter 1,

Please test the patch
04Commenter 4
Dear Commenter 1,
Branch CR28644 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested on Release mode.
[revision removed]

Number of compiler warnings:

occt component :
Linux: 0 (0 on master)
Windows: 0 (0 on master)
MacOS : 0 (0 on master)

products component :
Linux: 64 (64 on master)
Windows: 0 (0 on master)
MacOS : 1202

Regressions/Differences/Improvements:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
occt component :
Total MEMORY difference: 91763906 / 91371554 [+0.43%]
Total CPU difference: 19882.580000000435 / 19613.34000000024 [+1.37%]
products component :
Total MEMORY difference: 30816102 / 30770563 [+0.15%]
Total CPU difference: 5387.369999999989 / 5358.419999999984 [+0.54%]
Testing on Windows:
occt component :
Total MEMORY difference: 57857902 / 57855806 [+0.00%]
Total CPU difference: 18628.578613198544 / 18063.40259029849 [+3.13%]
products component :
Total MEMORY difference: 22295378 / 22257195 [+0.17%]
Total CPU difference: 5467.772649599969 / 5313.206858799958 [+2.91%]

There are no differences in images found by testdiff.
05Commenter 5
Dear Commenter 1,
Branch CR28644 is TESTED.
06Commenter 2
Branch [archived branch] has been deleted by Author.

[revision removed]

Related records