MantisBT
Mantis Bug Tracker Workflow

View Issue Details Jump to Notes ] Issue History ] Print ]
IDProjectCategoryView StatusDate SubmittedLast Update
0030687Open CASCADE[OCCT] OCCT:Visualizationpublic2019-05-01 14:122019-05-21 19:43
Reporterkgv 
Assigned Tobugmaster 
PrioritynormalSeverityintegration request 
StatustestedResolutionopen 
PlatformOSOS Version
Product Version 
Target Version[OCCT] 7.4.0*Fixed in Version 
Summary0030687: Visualization - remove redundant interfaces SelectBasics_EntityOwner and SelectBasics_SensitiveEntity
DescriptionSelectBasics_EntityOwner is a redundant and incomplete interface class having only single implementation - SelectMgr_EntityOwner.

Current design relies on DownCast() to SelectMgr_EntityOwner everywhere, hence eliminating redundant base class will make code more straightforward.
Steps To ReproduceN/A
TagsNo tags attached.
Test case numberNot required
Attached Files

- Relationships
related to 0030675verifiedbugmaster Visualization - remove redundant proxy classes in hierarchy of PrsMgr_Presentation 

-  Notes
(0084089)
git (administrator)
2019-05-01 14:21

Branch CR30687 has been created by kgv.

SHA-1: 78977a273341be2143a53fcdb1e98335cc053959


Detailed log of new commits:

Author: kgv
Date: Wed May 1 14:14:28 2019 +0300

    0030687: Visualization - remove redundant interface SelectBasics_EntityOwner
    
    SelectBasics_EntityOwner has been merged into SelectMgr_EntityOwner.
    Unused property SelectMgr_EntityOwner::ResetLocation() has been removed.
    SelectBasics package has been moved from TKService to TKV3d.
(0084091)
git (administrator)
2019-05-01 14:30

Branch CR30687 has been updated forcibly by kgv.

SHA-1: 01342afcc809bd937e6bf0ab8d6528fc04360f32
(0084092)
git (administrator)
2019-05-01 15:04

Branch CR30687 has been updated by kgv.

SHA-1: d436ecd29778840226a4fb3c093b8beb719e7471


Detailed log of new commits:

Author: kgv
Date: Wed May 1 14:58:07 2019 +0300

    SelectBasics_SensitiveEntity has been merged into Select3D_SensitiveEntity.

(0084093)
git (administrator)
2019-05-01 15:06

Branch CR30687_1 has been created by kgv.

SHA-1: e4d8a703fe9a454f32ceae65bd958ac250152e6d


Detailed log of new commits:

Author: kgv
Date: Wed May 1 14:14:28 2019 +0300

    0030687: Visualization - remove redundant interfaces SelectBasics_EntityOwner and SelectBasics_SensitiveEntity
    
    SelectBasics_EntityOwner has been merged into SelectMgr_EntityOwner.
    Unused property SelectMgr_EntityOwner::ResetLocation() has been removed.
    SelectBasics package has been moved from TKService to TKV3d.
    
    SelectBasics_SensitiveEntity has been merged into Select3D_SensitiveEntity.
(0084095)
git (administrator)
2019-05-01 15:49

Branch CR30687_1 has been updated forcibly by kgv.

SHA-1: f337e6589e0d27b39ed8c4c4ddb5e74362c82e6b
(0084099)
kgv (developer)
2019-05-01 23:08

Patch is ready for review.

http://jenkins-test-12.nnov.opencascade.com:8080/view/CR30687_1-CR30687_1-KGV [^]
(0084516)
san (developer)
2019-05-21 18:03

Branch CR30687_1 in occt and occt-products reviewed without remarks.
(0084523)
bugmaster (administrator)
2019-05-21 19:19

Combination -
OCCT branch : CR30687_1
master SHA - f337e6589e0d27b39ed8c4c4ddb5e74362c82e6b
d67d4b811012eef8913d3c535c29654d0acf3c4c
Products branch : CR30687_1 SHA - d5212caf37121bb7a36298ead5e0d5fd3f6b5b3e
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: 16326.55999999998 / 16286.290000000015 [+0.25%]
Products
Total CPU difference: 10536.410000000033 / 10492.700000000066 [+0.42%]
Windows-64-VC14:
OCCT
Total CPU difference: 17628.296875 / 17645.203125 [-0.10%]
Products
Total CPU difference: 12086.5 / 12107.4375 [-0.17%]


Image differences :
No differences that require special attention

Memory differences :
No differences that require special attention
(0084524)
git (administrator)
2019-05-21 19:43

Branch CR30687_2 has been created by kgv.

SHA-1: 0ef04197f700fdf9bb02c1d156c2a96753f4b838


Detailed log of new commits:

Author: kgv
Date: Tue May 21 19:42:15 2019 +0300

    0030687: Visualization - remove redundant interfaces SelectBasics_EntityOwner and SelectBasics_SensitiveEntity
    
    SelectBasics_EntityOwner has been merged into SelectMgr_EntityOwner.
    Unused property SelectMgr_EntityOwner::ResetLocation() has been removed.
    SelectBasics package has been moved from TKService to TKV3d.
    
    SelectBasics_SensitiveEntity has been merged into Select3D_SensitiveEntity.

- Issue History
Date Modified Username Field Change
2019-05-01 14:12 kgv New Issue
2019-05-01 14:12 kgv Assigned To => kgv
2019-05-01 14:12 kgv Relationship added related to 0030675
2019-05-01 14:21 git Note Added: 0084089
2019-05-01 14:30 git Note Added: 0084091
2019-05-01 15:04 git Note Added: 0084092
2019-05-01 15:05 kgv Summary Visualization - remove redundant interface SelectBasics_EntityOwner => Visualization - remove redundant interfaces SelectBasics_EntityOwner and SelectBasics_SensitiveEntity
2019-05-01 15:06 git Note Added: 0084093
2019-05-01 15:49 git Note Added: 0084095
2019-05-01 23:08 kgv Note Added: 0084099
2019-05-01 23:08 kgv Assigned To kgv => san
2019-05-01 23:08 kgv Status new => resolved
2019-05-21 18:03 san Note Added: 0084516
2019-05-21 18:03 san Assigned To san => bugmaster
2019-05-21 18:03 san Status resolved => reviewed
2019-05-21 19:17 bugmaster Test case number => Not required
2019-05-21 19:19 bugmaster Note Added: 0084523
2019-05-21 19:19 bugmaster Status reviewed => tested
2019-05-21 19:43 git Note Added: 0084524


Copyright © 2000 - 2019 MantisBT Team
Powered by Mantis Bugtracker