Anonymous | Login | 2021-01-19 01:17 MSK | ![]() |
My View | View Issues | Change Log | Roadmap |
View Issue Details [ Jump to Notes ] [ Related Changesets ] | [ Issue History ] [ Print ] | ||||||||
ID | Project | Category | View Status | Date Submitted | Last Update | ||||
0031067 | Community | [OCCT] OCCT:Visualization | public | 2019-10-14 13:40 | 2020-12-02 17:12 | ||||
Reporter | BenjaminBihler | ||||||||
Assigned To | apn | ||||||||
Priority | normal | Severity | minor | ||||||
Status | closed | Resolution | fixed | ||||||
Platform | A | OS | L | OS Version | L | ||||
Product Version | [OCCT] 7.4.0 | ||||||||
Target Version | [OCCT] 7.5.0 | Fixed in Version | [OCCT] 7.5.0 | ||||||
Summary | 0031067: Visualization - Aspect_Window::DoResize() should be a non-const method | ||||||||
Description | The method DoResize() of Aspect_Window is declared const. This makes it necessary to use a C-style cast or a const_cast to cast away the constness of the member fields myXLeft, ... myYBottom in OcctWindow::DoResize() because here the member fields are modified. Such code is surprising and weird. Depending on the context it might also be undefined behaviour. Wouldn't it be better to make Aspect_Window::DoResize() a non-const method? | ||||||||
Tags | No tags attached. | ||||||||
Test case number | Not needed | ||||||||
Attached Files | |||||||||
![]() |
|
(0088142) git (administrator) 2019-10-15 20:36 |
Branch CR31067 has been created by kgv. SHA-1: 81abbff06ec4d8bea605634a54bb72082e8a7279 Detailed log of new commits: Author: kgv Date: Tue Oct 15 20:36:06 2019 +0300 0031067: Visualization - Aspect_Window::DoResize() should be a non-const method Removed 'const' from Aspect_Window::DoResize() virtual method. |
(0088146) git (administrator) 2019-10-15 20:51 |
Branch CR31067 has been updated forcibly by kgv. SHA-1: d50dc0689050a96ea7ceb0164046b4eaeb4c59ed |
(0088149) kgv (developer) 2019-10-15 23:52 |
Patch is ready for review. |
(0088156) abv (manager) 2019-10-16 08:43 |
Good change; but should not we describe it in Upgrade Guide? |
(0088158) git (administrator) 2019-10-16 10:43 |
Branch CR31067 has been updated forcibly by kgv. SHA-1: 04bd4f403cd62dc36501f4527b5413ef1747afef |
(0088159) kgv (developer) 2019-10-16 10:43 |
> should not we describe it in Upgrade Guide? Added. |
(0088181) abv (manager) 2019-10-16 16:25 |
Reviewed, please integrate |
(0088488) git (administrator) 2019-10-23 11:51 |
Branch CR31067 has been deleted by kgv. SHA-1: 04bd4f403cd62dc36501f4527b5413ef1747afef |
![]() |
|||
occt: master 1a5007a9
Timestamp: 2019-10-19 12:38:48 Author: kgv Committer: apn [ Details ] [ Diff ] |
0031067: Visualization - Aspect_Window::DoResize() should be a non-const method Removed 'const' from Aspect_Window::DoResize() virtual method. |
||
mod - dox/dev_guides/upgrade/upgrade.md | [ Diff ] [ File ] | ||
mod - samples/glfw/GlfwOcctWindow.cpp | [ Diff ] [ File ] | ||
mod - samples/glfw/GlfwOcctWindow.h | [ Diff ] [ File ] | ||
mod - samples/java/jniviewer/jni/OcctJni_Window.hxx | [ Diff ] [ File ] | ||
mod - samples/qt/AndroidQt/src/AndroidQt_Window.h | [ Diff ] [ File ] | ||
mod - samples/qt/Common/src/OcctWindow.cxx | [ Diff ] [ File ] | ||
mod - samples/qt/Common/src/OcctWindow.h | [ Diff ] [ File ] | ||
mod - src/Aspect/Aspect_NeutralWindow.hxx | [ Diff ] [ File ] | ||
mod - src/Aspect/Aspect_Window.hxx | [ Diff ] [ File ] | ||
mod - src/Cocoa/Cocoa_Window.hxx | [ Diff ] [ File ] | ||
mod - src/Cocoa/Cocoa_Window.mm | [ Diff ] [ File ] | ||
mod - src/WNT/WNT_Window.cxx | [ Diff ] [ File ] | ||
mod - src/WNT/WNT_Window.hxx | [ Diff ] [ File ] | ||
mod - src/Xw/Xw_Window.cxx | [ Diff ] [ File ] | ||
mod - src/Xw/Xw_Window.hxx | [ Diff ] [ File ] | ||
![]() |
|||
Date Modified | Username | Field | Change |
2019-10-14 13:40 | BenjaminBihler | New Issue | |
2019-10-14 13:40 | BenjaminBihler | Assigned To | => kgv |
2019-10-15 20:36 | git | Note Added: 0088142 | |
2019-10-15 20:51 | git | Note Added: 0088146 | |
2019-10-15 23:52 | kgv | Note Added: 0088149 | |
2019-10-15 23:52 | kgv | Assigned To | kgv => abv |
2019-10-15 23:52 | kgv | Status | new => resolved |
2019-10-16 08:43 | abv | Note Added: 0088156 | |
2019-10-16 08:43 | abv | Assigned To | abv => kgv |
2019-10-16 08:43 | abv | Status | resolved => feedback |
2019-10-16 10:43 | git | Note Added: 0088158 | |
2019-10-16 10:43 | kgv | Note Added: 0088159 | |
2019-10-16 10:43 | kgv | Assigned To | kgv => abv |
2019-10-16 10:43 | kgv | Status | feedback => resolved |
2019-10-16 16:25 | abv | Note Added: 0088181 | |
2019-10-16 16:25 | abv | Assigned To | abv => bugmaster |
2019-10-16 16:25 | abv | Status | resolved => reviewed |
2019-10-23 01:07 | apn | Changeset attached | => occt master 1a5007a9 |
2019-10-23 01:07 | apn | Assigned To | bugmaster => apn |
2019-10-23 01:07 | apn | Status | reviewed => verified |
2019-10-23 01:07 | apn | Resolution | open => fixed |
2019-10-23 11:51 | git | Note Added: 0088488 | |
2019-10-23 16:52 | apn | Test case number | => Not needed |
2020-12-02 16:40 | emo | Fixed in Version | => 7.5.0 |
2020-12-02 17:12 | emo | Status | verified => closed |
Copyright © 2000 - 2021 MantisBT Team |