View Issue Details

IDProjectCategoryView StatusLast Update
0025094Open CASCADEOCCT:Codingpublic2014-11-11 12:53
Reporterkgv Assigned Tobugmaster  
PrioritynormalSeverityintegration request 
Status closedResolutionfixed 
Product Version6.7.1 
Target Version6.8.0Fixed in Version6.8.0 
Summary0025094: Visualization, Visual3d_View - code clean up
DescriptionCode of Visual3d_View class is bloated with debugging sections, never reached conditions, and obsolete solutions escaped by macros.
TagsNo tags attached.
Test case numberNot needed

Relationships

parent of 0024776 closedkgv Visualization - inherit OpenGl_View from Graphic3d_CView 

Activities

git

2014-07-20 17:48

administrator   ~0030256

Branch CR25094 has been created by kgv.

SHA-1: c596d0047a1c4ec808af4b06fb4cdddb4a2da32a

kgv

2014-07-20 17:49

developer   ~0030257

Patch is ready for review in branch CR25094.

san

2014-07-21 20:54

developer   ~0030281

Branch CR25094 reviewed without remarks, ready for testing.

mkv

2014-07-23 12:42

tester   ~0030377

Dear BugMaster,

Branch CR25094 from occt git-repository (and master from products git-repository) was compiled on Linux, MacOS and Windows platforms and tested.
SHA-1: c596d0047a1c4ec808af4b06fb4cdddb4a2da32a

Number of compiler warnings:

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

products component :
Linux: 11 (11 on master)
Windows: 1 (1 on master)

Regressions/Differences:
No regressions/differences

Testing cases:
Not needed

Testing on Linux:
Total MEMORY difference: 352377408 / 351969036
Total CPU difference: 47487.31000000008 / 46305.26

Testing on Windows:
Total MEMORY difference: 239469424 / 239178976
Total CPU difference: 34705.609375 / 35090.328125

There are no differences in images found by testdiff.

git

2014-07-28 16:42

administrator   ~0030444

Branch CR25094 has been deleted by inv.

SHA-1: c596d0047a1c4ec808af4b06fb4cdddb4a2da32a

Related Changesets

occt: master d4f1753b

2014-07-20 11:04:13

kgv


Committer: bugmaster Details Diff
0025094: Visualization, Visual3d_View - code clean up

Remove debug and obsolete code. Apply coding rules.
Affected Issues
0025094
mod - src/Visual3d/FILES Diff File
mod - src/Visual3d/Visual3d_View.cdl Diff File
mod - src/Visual3d/Visual3d_View.cxx Diff File
rm - src/Visual3d/Visual3d_View.pxx Diff File
mod - src/Visual3d/Visual3d_ViewManager.cxx Diff File
rm - src/Visual3d/Visual3d_ViewManager.pxx Diff File
rm - src/Visual3d/Visual3d_View_Print.cxx Diff File

Issue History

Date Modified Username Field Change
2014-07-20 15:09 kgv New Issue
2014-07-20 15:09 kgv Assigned To => kgv
2014-07-20 15:09 kgv Relationship added parent of 0024776
2014-07-20 17:48 git Note Added: 0030256
2014-07-20 17:49 kgv Note Added: 0030257
2014-07-20 17:49 kgv Assigned To kgv => san
2014-07-20 17:49 kgv Status new => resolved
2014-07-21 20:54 san Note Added: 0030281
2014-07-21 20:54 san Assigned To san => bugmaster
2014-07-21 20:54 san Status resolved => reviewed
2014-07-22 12:51 mkv Assigned To bugmaster => mkv
2014-07-23 12:42 mkv Note Added: 0030377
2014-07-23 12:43 mkv Test case number => Not needed
2014-07-23 12:43 mkv Assigned To mkv => bugmaster
2014-07-23 12:43 mkv Status reviewed => tested
2014-07-25 14:45 bugmaster Changeset attached => occt master d4f1753b
2014-07-25 14:45 bugmaster Status tested => verified
2014-07-25 14:45 bugmaster Resolution open => fixed
2014-07-28 16:42 git Note Added: 0030444
2014-11-11 12:46 aiv Fixed in Version => 6.8.0
2014-11-11 12:53 aiv Status verified => closed