Archived issue #0032160
Coding Rules - merge AIS_InteractiveContext_*.cxx into single file
Description
AIS_InteractiveContext implementation is historically split into several .cxx files.
There is no strong logic behind this splitting - functionality is practically spread around all .cxx files complicating source code navigation.
It is proposed merging these files together.
There is no strong logic behind this splitting - functionality is practically spread around all .cxx files complicating source code navigation.
It is proposed merging these files together.
Steps to reproduce
N/A
Public activity
7 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Author.
[revision removed]
Detailed log of new commits:
Author: Author
Date: Wed Feb 24 20:03:40 2021 +0300
0032160: Coding Rules - merge AIS_InteractiveContext_*.cxx into single file
Removed redundant .lxx files from API package.
AIS_InteractiveContext_1.cxx,AIS_InteractiveContext_2.cxx,AIS_InteractiveContext_3.cxx
have been merged into [archived branch]
V3d_View_2.cxx,V3d_View_3.cxx,V3d_View_4.cxx have been merged into [archived branch]
V3d_View_4.cxx has been merged into [archived branch]
[revision removed]
Detailed log of new commits:
Author: Author
Date: Wed Feb 24 20:03:40 2021 +0300
0032160: Coding Rules - merge AIS_InteractiveContext_*.cxx into single file
Removed redundant .lxx files from API package.
AIS_InteractiveContext_1.cxx,AIS_InteractiveContext_2.cxx,AIS_InteractiveContext_3.cxx
have been merged into [archived branch]
V3d_View_2.cxx,V3d_View_3.cxx,V3d_View_4.cxx have been merged into [archived branch]
V3d_View_4.cxx has been merged into [archived branch]
Patch is ready for review
- OCCT: branch CR32160.
http://jenkins-test-12.nnov.opencascade.com:8080/view/CR32160-master-KGV/
- OCCT: branch CR32160.
http://jenkins-test-12.nnov.opencascade.com:8080/view/CR32160-master-KGV/
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
The patch was reviewed
Branch [archived branch] has been updated forcibly by Author.
[revision removed]
[revision removed]
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: 17811.770000000153 / 17875.850000000235 [-0.36%]
Products
Total CPU difference: 11543.6500000001 / 11549.600000000113 [-0.05%]
Windows-64-VC14:
OCCT
Total CPU difference: 19344.3125 / 19351.640625 [-0.04%]
Products
Total CPU difference: 12900.578125 / 12845.234375 [+0.43%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
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: 17811.770000000153 / 17875.850000000235 [-0.36%]
Products
Total CPU difference: 11543.6500000001 / 11549.600000000113 [-0.05%]
Windows-64-VC14:
OCCT
Total CPU difference: 19344.3125 / 19351.640625 [-0.04%]
Products
Total CPU difference: 12900.578125 / 12845.234375 [+0.43%]
Image differences :
No differences that require special attention
Memory differences :
No differences that require special attention
Branch [archived branch] has been deleted by Participant.
[revision removed]
[revision removed]
Related records