Archived issue #0028664
Documentation of Graphic3d_CView::Activate and Deactivate does not seem to be correct
Description
The doxygen documentation of Graphic3d_CView::Activate says:
"Activates the view. Map the associated window on the screen and post the view in this window. Warning: Raises ViewDefinitionError if the associated window isn't defined."
When looking at the code, it doesn't seem to deal with the window.
"Activates the view. Map the associated window on the screen and post the view in this window. Warning: Raises ViewDefinitionError if the associated window isn't defined."
When looking at the code, it doesn't seem to deal with the window.
Public activity
10 archived notes
Participants are labeled by their role within this record.
This is an artifact of removed class Visual3d_View.
Branch [archived branch] has been created by Commenter 2.
[revision removed]
Detailed log of new commits:
Author: Commenter 2
Date: Tue May 2 10:22:59 2017 +0300
0028664: Documentation of Graphic3d_CView::Activate and Deactivate does not seem to be correct
[revision removed]
Detailed log of new commits:
Author: Commenter 2
Date: Tue May 2 10:22:59 2017 +0300
0028664: Documentation of Graphic3d_CView::Activate and Deactivate does not seem to be correct
Patch is ready for review.
Branch CR28664 reviewed without remarks, ready for testing.
Dear ysn,
could you please review.
could you please review.
Branch [archived branch] has been updated by Participant.
[revision removed]
Detailed log of new commits:
Author: ysn
Date: Wed May 17 12:46:51 2017 +0300
Minor grammar changes.
[revision removed]
Detailed log of new commits:
Author: ysn
Date: Wed May 17 12:46:51 2017 +0300
Minor grammar changes.
reviewed and corrected
> Author: ysn
> Date: Wed May 17 12:46:51 2017 +0300
> Minor grammar changes.
Hint - I suppose the commit should be:
> // Minor grammar changes.
indicating that the message should be skipped while integrating into master branch (as discussed on last meeting).
> Date: Wed May 17 12:46:51 2017 +0300
> Minor grammar changes.
Hint - I suppose the commit should be:
> // Minor grammar changes.
indicating that the message should be skipped while integrating into master branch (as discussed on last meeting).
Dear Commenter 1,
Branch CR28664 is TESTED.
Branch CR28664 is TESTED.
Branch [archived branch] has been deleted by Commenter 2.
[revision removed]
[revision removed]
Related records