Archived issue #0029419
Make V3d_Viewer::PrivilegedPlane() return const reference rather than a temp object
Description
The declare of of PrivilegedPlane in class V3d_Viewer as below:
gp_Ax3 PrivilegedPlane() const;
It's better to change it to:
inline const gp_Ax3& PrivilegedPlane() const {return myPrivilegedPlane;}
I have one more question, why not use gp_Ax2 as PrivilegedPlane? is it need to use left hand coordinate system?
gp_Ax3 PrivilegedPlane() const;
It's better to change it to:
inline const gp_Ax3& PrivilegedPlane() const {return myPrivilegedPlane;}
I have one more question, why not use gp_Ax2 as PrivilegedPlane? is it need to use left hand coordinate system?
Public activity
4 archived notes
Participants are labeled by their role within this record.
Branch [archived branch] has been created by Commenter 3.
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Fri Jan 12 10:36:18 2018 +0300
0029419: Make V3d_Viewer::PrivilegedPlane() return const reference rather than a temp object
[revision removed]
Detailed log of new commits:
Author: Commenter 3
Date: Fri Jan 12 10:36:18 2018 +0300
0029419: Make V3d_Viewer::PrivilegedPlane() return const reference rather than a temp object
Combination -
OCCT branch : [archived branch] SHA - [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
OCCT branch : [archived branch] SHA - [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
Branch [archived branch] has been deleted by Commenter 3.
[revision removed]
[revision removed]