Ros-diffs

ros-diffs@reactos.org

81390 discussions

[reactos] 02/03: [NTOS:MM] Simplify definitions of MM_SYSLDR_NO_IMPORTS and MM_SYSLDR_BOOT_LOADED.
by Hermès Bélusca-Maïto 18 Mar '23

18 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a8bcc8d7a1551c47bdacd… commit a8bcc8d7a1551c47bdacdf544b89df9b3679aa7a Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Wed Mar 15 20:31:28 2023 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Sat Mar 18 03:42:31 2023 +0100 [NTOS:MM] Simplify definitions of MM_SYSLDR_NO_IMPORTS and MM_SYSLDR_BOOT_LOADED. --- ntoskrnl/mm/ARM3/miarm.h | 9 ++------- 1 file changed, 2 insertions(+), 7 deletions(-) diff --git a/ntoskrnl/mm/ARM3/miarm.h b/ntoskrnl/mm/ARM3/miarm.h index 60197cda74e..1776b69a918 100644 --- a/ntoskrnl/mm/ARM3/miarm.h +++ b/ntoskrnl/mm/ARM3/miarm.h @@ -197,13 +197,8 @@ extern const ULONG MmProtectToValue[32]; // // Special values for LoadedImports // -#ifdef _WIN64 -#define MM_SYSLDR_NO_IMPORTS (PVOID)0xFFFFFFFFFFFFFFFEULL -#define MM_SYSLDR_BOOT_LOADED (PVOID)0xFFFFFFFFFFFFFFFFULL -#else -#define MM_SYSLDR_NO_IMPORTS (PVOID)0xFFFFFFFE -#define MM_SYSLDR_BOOT_LOADED (PVOID)0xFFFFFFFF -#endif +#define MM_SYSLDR_NO_IMPORTS ((PVOID)(ULONG_PTR)-2) +#define MM_SYSLDR_BOOT_LOADED ((PVOID)(ULONG_PTR)-1) #define MM_SYSLDR_SINGLE_ENTRY 0x1 //

1 0

[reactos] 01/03: [SDK] driverspecs.h: Add missing dummy _IRQL_limited_to_() definition (when _PREFAST_ is not defined).
by Hermès Bélusca-Maïto 18 Mar '23

18 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=f6c8155239a6f402c6b39… commit f6c8155239a6f402c6b3905d08a51f0befe877a2 Author: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> AuthorDate: Sat Mar 18 03:06:54 2023 +0100 Commit: Hermès Bélusca-Maïto <hermes.belusca-maito(a)reactos.org> CommitDate: Sat Mar 18 03:42:26 2023 +0100 [SDK] driverspecs.h: Add missing dummy _IRQL_limited_to_() definition (when _PREFAST_ is not defined). Addendum to commit 8a688204. --- sdk/include/psdk/driverspecs.h | 1 + 1 file changed, 1 insertion(+) diff --git a/sdk/include/psdk/driverspecs.h b/sdk/include/psdk/driverspecs.h index b440566d460..5072d8430d3 100644 --- a/sdk/include/psdk/driverspecs.h +++ b/sdk/include/psdk/driverspecs.h @@ -235,6 +235,7 @@ __PRIMOP(int, _Is_driver_(void);) #define _IRQL_saves_ #define _IRQL_saves_global_(kind,param) #define _IRQL_uses_cancel_ +#define _IRQL_limited_to_(level) #define _Kernel_clear_do_init_(yesNo) #define _Kernel_float_restored_ #define _Kernel_float_saved_

1 0

[reactos] 01/01: [MSPAINT] Encapsulation: hNontranspIcon and hTranspIcon
by Katayama Hirofumi MZ 17 Mar '23

17 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=e9f3790aa4f1a1a211c6d… commit e9f3790aa4f1a1a211c6dbbfa68c2e955fe7cda6 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sat Mar 18 08:24:19 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Sat Mar 18 08:24:19 2023 +0900 [MSPAINT] Encapsulation: hNontranspIcon and hTranspIcon CORE-18867 --- base/applications/mspaint/main.cpp | 6 ------ base/applications/mspaint/toolsettings.cpp | 17 +++++++++++++++-- base/applications/mspaint/toolsettings.h | 4 ++++ 3 files changed, 19 insertions(+), 8 deletions(-) diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index bd83e56a04a..2df191af3c8 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -192,12 +192,6 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument SetMenu(hwnd, menu); haccel = LoadAccelerators(hThisInstance, MAKEINTRESOURCE(800)); - /* preloading the draw transparent/nontransparent icons for later use */ - hNontranspIcon = - (HICON) LoadImage(hThisInstance, MAKEINTRESOURCE(IDI_NONTRANSPARENT), IMAGE_ICON, 40, 30, LR_DEFAULTCOLOR); - hTranspIcon = - (HICON) LoadImage(hThisInstance, MAKEINTRESOURCE(IDI_TRANSPARENT), IMAGE_ICON, 40, 30, LR_DEFAULTCOLOR); - RECT toolBoxContainerPos = {2, 2, 2 + 52, 2 + 350}; toolBoxContainer.Create(hwnd, toolBoxContainerPos, NULL, WS_CHILD | WS_VISIBLE); /* creating the tool settings child window */ diff --git a/base/applications/mspaint/toolsettings.cpp b/base/applications/mspaint/toolsettings.cpp index 19f826ea825..dc1883bf47e 100644 --- a/base/applications/mspaint/toolsettings.cpp +++ b/base/applications/mspaint/toolsettings.cpp @@ -15,6 +15,12 @@ LRESULT CToolSettingsWindow::OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, WINBOOL& bHandled) { + /* preloading the draw transparent/nontransparent icons for later use */ + m_hNontranspIcon = (HICON)LoadImage(hProgInstance, MAKEINTRESOURCE(IDI_NONTRANSPARENT), + IMAGE_ICON, 40, 30, LR_DEFAULTCOLOR); + m_hTranspIcon = (HICON)LoadImage(hProgInstance, MAKEINTRESOURCE(IDI_TRANSPARENT), + IMAGE_ICON, 40, 30, LR_DEFAULTCOLOR); + RECT trackbarZoomPos = {1, 1, 1 + 40, 1 + 64}; trackbarZoom.Create(TRACKBAR_CLASS, m_hWnd, trackbarZoomPos, NULL, WS_CHILD | TBS_VERT | TBS_AUTOTICKS); trackbarZoom.SendMessage(TBM_SETRANGE, (WPARAM) TRUE, MAKELPARAM(0, 6)); @@ -22,6 +28,13 @@ LRESULT CToolSettingsWindow::OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, W return 0; } +LRESULT CToolSettingsWindow::OnDestroy(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +{ + ::DestroyIcon(m_hNontranspIcon); + ::DestroyIcon(m_hTranspIcon); + return 0; +} + LRESULT CToolSettingsWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { if (!zoomTo(125 << trackbarZoom.SendMessage(TBM_GETPOS, 0, 0), 0, 0)) @@ -50,8 +63,8 @@ LRESULT CToolSettingsWindow::OnPaint(UINT nMsg, WPARAM wParam, LPARAM lParam, BO SelectObject(hdc, GetSysColorBrush(COLOR_HIGHLIGHT)); Rectangle(hdc, 2, toolsModel.IsBackgroundTransparent() * 31 + 2, 41, toolsModel.IsBackgroundTransparent() * 31 + 33); DeleteObject(SelectObject(hdc, oldPen)); - DrawIconEx(hdc, 1, 2, hNontranspIcon, 40, 30, 0, NULL, DI_NORMAL); - DrawIconEx(hdc, 1, 33, hTranspIcon, 40, 30, 0, NULL, DI_NORMAL); + DrawIconEx(hdc, 1, 2, m_hNontranspIcon, 40, 30, 0, NULL, DI_NORMAL); + DrawIconEx(hdc, 1, 33, m_hTranspIcon, 40, 30, 0, NULL, DI_NORMAL); break; } case TOOL_RUBBER: diff --git a/base/applications/mspaint/toolsettings.h b/base/applications/mspaint/toolsettings.h index 4b06cb7960b..bd92d61a6ac 100644 --- a/base/applications/mspaint/toolsettings.h +++ b/base/applications/mspaint/toolsettings.h @@ -21,11 +21,15 @@ public: MESSAGE_HANDLER(WM_TOOLSMODELTOOLCHANGED, OnToolsModelToolChanged) MESSAGE_HANDLER(WM_TOOLSMODELSETTINGSCHANGED, OnToolsModelSettingsChanged) MESSAGE_HANDLER(WM_TOOLSMODELZOOMCHANGED, OnToolsModelZoomChanged) + MESSAGE_HANDLER(WM_DESTROY, OnDestroy) END_MSG_MAP() CWindow trackbarZoom; + HICON m_hNontranspIcon; + HICON m_hTranspIcon; LRESULT OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); + LRESULT OnDestroy(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnPaint(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled);

1 0

[reactos] 01/01: [MSPAINT] Encapsulation: hCurFill, hCurColor etc.
by Katayama Hirofumi MZ 17 Mar '23

17 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=cc3b53fef9892ea30b655… commit cc3b53fef9892ea30b655e851a27cd1f4a2e0898 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Sat Mar 18 08:09:26 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Sat Mar 18 08:09:26 2023 +0900 [MSPAINT] Encapsulation: hCurFill, hCurColor etc. CORE-18867 --- base/applications/mspaint/globalvar.h | 6 ------ base/applications/mspaint/imgarea.cpp | 25 +++++++++++++++++-------- base/applications/mspaint/imgarea.h | 8 ++++++++ base/applications/mspaint/main.cpp | 12 ------------ 4 files changed, 25 insertions(+), 26 deletions(-) diff --git a/base/applications/mspaint/globalvar.h b/base/applications/mspaint/globalvar.h index aa144275968..b0099e571f5 100644 --- a/base/applications/mspaint/globalvar.h +++ b/base/applications/mspaint/globalvar.h @@ -36,12 +36,6 @@ extern OPENFILENAME sfn; extern HICON hNontranspIcon; extern HICON hTranspIcon; -extern HCURSOR hCurFill; -extern HCURSOR hCurColor; -extern HCURSOR hCurZoom; -extern HCURSOR hCurPen; -extern HCURSOR hCurAirbrush; - extern HINSTANCE hProgInstance; extern TCHAR filepathname[1000]; diff --git a/base/applications/mspaint/imgarea.cpp b/base/applications/mspaint/imgarea.cpp index 1d8b05024ce..aa116dcc8c1 100644 --- a/base/applications/mspaint/imgarea.cpp +++ b/base/applications/mspaint/imgarea.cpp @@ -8,12 +8,21 @@ * Katayama Hirofumi MZ */ -/* INCLUDES *********************************************************/ - #include "precomp.h" /* FUNCTIONS ********************************************************/ +LRESULT CImgAreaWindow::OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +{ + m_hCurFill = LoadIcon(hProgInstance, MAKEINTRESOURCE(IDC_FILL)); + m_hCurColor = LoadIcon(hProgInstance, MAKEINTRESOURCE(IDC_COLOR)); + m_hCurZoom = LoadIcon(hProgInstance, MAKEINTRESOURCE(IDC_ZOOM)); + m_hCurPen = LoadIcon(hProgInstance, MAKEINTRESOURCE(IDC_PEN)); + m_hCurAirbrush = LoadIcon(hProgInstance, MAKEINTRESOURCE(IDC_AIRBRUSH)); + + return 0; +} + void CImgAreaWindow::drawZoomFrame(int mouseX, int mouseY) { HDC hdc; @@ -115,22 +124,22 @@ LRESULT CImgAreaWindow::OnSetCursor(UINT nMsg, WPARAM wParam, LPARAM lParam, BOO switch (toolsModel.GetActiveTool()) { case TOOL_FILL: - SetCursor(hCurFill); + ::SetCursor(m_hCurFill); break; case TOOL_COLOR: - SetCursor(hCurColor); + ::SetCursor(m_hCurColor); break; case TOOL_ZOOM: - SetCursor(hCurZoom); + ::SetCursor(m_hCurZoom); break; case TOOL_PEN: - SetCursor(hCurPen); + ::SetCursor(m_hCurPen); break; case TOOL_AIRBRUSH: - SetCursor(hCurAirbrush); + ::SetCursor(m_hCurAirbrush); break; default: - SetCursor(LoadCursor(NULL, IDC_CROSS)); + ::SetCursor(::LoadCursor(NULL, IDC_CROSS)); } return 0; } diff --git a/base/applications/mspaint/imgarea.h b/base/applications/mspaint/imgarea.h index 306c94088e9..d7b44e5c4af 100644 --- a/base/applications/mspaint/imgarea.h +++ b/base/applications/mspaint/imgarea.h @@ -42,9 +42,17 @@ public: MESSAGE_HANDLER(WM_CAPTURECHANGED, OnCaptureChanged) MESSAGE_HANDLER(WM_KEYDOWN, OnKeyDown) MESSAGE_HANDLER(WM_CTLCOLOREDIT, OnCtlColorEdit) + MESSAGE_HANDLER(WM_CREATE, OnCreate) END_MSG_MAP() private: + HCURSOR m_hCurFill; + HCURSOR m_hCurColor; + HCURSOR m_hCurZoom; + HCURSOR m_hCurPen; + HCURSOR m_hCurAirbrush; + + LRESULT OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnEraseBkGnd(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnPaint(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index a1f11a87af9..bd83e56a04a 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -35,12 +35,6 @@ OPENFILENAME sfn; HICON hNontranspIcon; HICON hTranspIcon; -HCURSOR hCurFill; -HCURSOR hCurColor; -HCURSOR hCurZoom; -HCURSOR hCurPen; -HCURSOR hCurAirbrush; - HINSTANCE hProgInstance; TCHAR filepathname[1000]; @@ -204,12 +198,6 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument hTranspIcon = (HICON) LoadImage(hThisInstance, MAKEINTRESOURCE(IDI_TRANSPARENT), IMAGE_ICON, 40, 30, LR_DEFAULTCOLOR); - hCurFill = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_FILL)); - hCurColor = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_COLOR)); - hCurZoom = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_ZOOM)); - hCurPen = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_PEN)); - hCurAirbrush = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_AIRBRUSH)); - RECT toolBoxContainerPos = {2, 2, 2 + 52, 2 + 350}; toolBoxContainer.Create(hwnd, toolBoxContainerPos, NULL, WS_CHILD | WS_VISIBLE); /* creating the tool settings child window */

1 0

[reactos] 01/01: [MSPAINT] Add Static Edge to toolbars
by Katayama Hirofumi MZ 17 Mar '23

17 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=d40de8f47c955bd80310a… commit d40de8f47c955bd80310a2ec6715825637e845a2 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 19:18:07 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Fri Mar 17 19:18:07 2023 +0900 [MSPAINT] Add Static Edge to toolbars - Add WS_EX_STATICEDGE to paletteWindow. - Remove CCS_NODIVIDER from toolbar. CORE-18867 --- base/applications/mspaint/main.cpp | 2 +- base/applications/mspaint/toolbox.cpp | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index aabe1992502..a1f11a87af9 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -218,7 +218,7 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument /* creating the palette child window */ RECT paletteWindowPos = {56, 9, 56 + 255, 9 + 32}; - paletteWindow.Create(hwnd, paletteWindowPos, NULL, WS_CHILD | WS_VISIBLE); + paletteWindow.Create(hwnd, paletteWindowPos, NULL, WS_CHILD | WS_VISIBLE, WS_EX_STATICEDGE); // creating the canvas RECT canvasWindowPos = {0, 0, 0 + 500, 0 + 500}; diff --git a/base/applications/mspaint/toolbox.cpp b/base/applications/mspaint/toolbox.cpp index c989b882295..4af48b5917a 100644 --- a/base/applications/mspaint/toolbox.cpp +++ b/base/applications/mspaint/toolbox.cpp @@ -23,7 +23,7 @@ LRESULT CToolBox::OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandl /* NOTE: The horizontal line above the toolbar is hidden by CCS_NODIVIDER style. */ RECT toolbarPos = {0, 0, CX_TOOLBAR, CY_TOOLBAR}; DWORD style = WS_CHILD | WS_VISIBLE | CCS_NOPARENTALIGN | CCS_VERT | CCS_NORESIZE | - TBSTYLE_TOOLTIPS | TBSTYLE_FLAT | CCS_NODIVIDER; + TBSTYLE_TOOLTIPS | TBSTYLE_FLAT; toolbar.Create(TOOLBARCLASSNAME, m_hWnd, toolbarPos, NULL, style); hImageList = ImageList_Create(16, 16, ILC_COLOR24 | ILC_MASK, 16, 0); toolbar.SendMessage(TB_SETIMAGELIST, 0, (LPARAM) hImageList);

1 0

[reactos] 01/01: [MSPAINT] Delete useless "STATIC" control
by Katayama Hirofumi MZ 17 Mar '23

17 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=868c7c4d53fdbea8fdbcd… commit 868c7c4d53fdbea8fdbcd6ecf01e9cff71550a5a Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 19:01:42 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Fri Mar 17 19:01:42 2023 +0900 [MSPAINT] Delete useless "STATIC" control CORE-18867 --- base/applications/mspaint/main.cpp | 3 --- 1 file changed, 3 deletions(-) diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index 75cc5862110..aabe1992502 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -210,9 +210,6 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument hCurPen = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_PEN)); hCurAirbrush = LoadIcon(hThisInstance, MAKEINTRESOURCE(IDC_AIRBRUSH)); - CreateWindowEx(0, _T("STATIC"), NULL, WS_CHILD | WS_VISIBLE | SS_ETCHEDHORZ, 0, 0, 5000, 2, hwnd, NULL, - hThisInstance, NULL); - RECT toolBoxContainerPos = {2, 2, 2 + 52, 2 + 350}; toolBoxContainer.Create(hwnd, toolBoxContainerPos, NULL, WS_CHILD | WS_VISIBLE); /* creating the tool settings child window */

1 0

[reactos] 01/01: [MSPAINT] Move UpdateCanvas's code to newly-defined CCanvasWindow::Update
by Katayama Hirofumi MZ 16 Mar '23

16 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=eacf003c816335e79ab58… commit eacf003c816335e79ab583f1af7a900aef0642c5 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 07:51:12 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Fri Mar 17 07:51:12 2023 +0900 [MSPAINT] Move UpdateCanvas's code to newly-defined CCanvasWindow::Update CORE-18867 --- base/applications/mspaint/canvas.cpp | 25 ++++++++++++------------- base/applications/mspaint/canvas.h | 5 +++-- base/applications/mspaint/imgarea.cpp | 4 ++-- 3 files changed, 17 insertions(+), 17 deletions(-) diff --git a/base/applications/mspaint/canvas.cpp b/base/applications/mspaint/canvas.cpp index 1293471ddbf..49249373ab8 100644 --- a/base/applications/mspaint/canvas.cpp +++ b/base/applications/mspaint/canvas.cpp @@ -13,23 +13,22 @@ /* FUNCTIONS ********************************************************/ -void -UpdateCanvas(HWND hwndFrom) +VOID CCanvasWindow::Update(HWND hwndFrom) { CRect tempRect; - canvasWindow.GetClientRect(&tempRect); + GetClientRect(&tempRect); CSize sizeScrollBox(tempRect.Width(), tempRect.Height()); CSize sizeZoomed = { Zoomed(imageModel.GetWidth()), Zoomed(imageModel.GetHeight()) }; CSize sizeWhole = { sizeZoomed.cx + (GRIP_SIZE * 2), sizeZoomed.cy + (GRIP_SIZE * 2) }; /* show/hide the scrollbars */ - canvasWindow.ShowScrollBar(SB_HORZ, sizeScrollBox.cx < sizeWhole.cx); - canvasWindow.ShowScrollBar(SB_VERT, sizeScrollBox.cy < sizeWhole.cy); + ShowScrollBar(SB_HORZ, sizeScrollBox.cx < sizeWhole.cx); + ShowScrollBar(SB_VERT, sizeScrollBox.cy < sizeWhole.cy); if (sizeScrollBox.cx < sizeWhole.cx || sizeScrollBox.cy < sizeWhole.cy) { - canvasWindow.GetClientRect(&tempRect); + GetClientRect(&tempRect); sizeScrollBox = CSize(tempRect.Width(), tempRect.Height()); } @@ -38,14 +37,14 @@ UpdateCanvas(HWND hwndFrom) si.nMax = sizeWhole.cx; si.nPage = sizeScrollBox.cx; - canvasWindow.SetScrollInfo(SB_HORZ, &si); + SetScrollInfo(SB_HORZ, &si); si.nMax = sizeWhole.cy; si.nPage = sizeScrollBox.cy; - canvasWindow.SetScrollInfo(SB_VERT, &si); + SetScrollInfo(SB_VERT, &si); - INT dx = -canvasWindow.GetScrollPos(SB_HORZ); - INT dy = -canvasWindow.GetScrollPos(SB_VERT); + INT dx = -GetScrollPos(SB_HORZ); + INT dy = -GetScrollPos(SB_VERT); if (sizeboxLeftTop.IsWindow()) { @@ -81,7 +80,7 @@ LRESULT CCanvasWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHa { if (m_hWnd) { - UpdateCanvas(m_hWnd); + Update(m_hWnd); } return 0; } @@ -112,7 +111,7 @@ LRESULT CCanvasWindow::OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& break; } SetScrollInfo(SB_HORZ, &si); - UpdateCanvas(m_hWnd); + Update(m_hWnd); return 0; } @@ -142,7 +141,7 @@ LRESULT CCanvasWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& break; } SetScrollInfo(SB_VERT, &si); - UpdateCanvas(m_hWnd); + Update(m_hWnd); return 0; } diff --git a/base/applications/mspaint/canvas.h b/base/applications/mspaint/canvas.h index b97d43a497d..22613518857 100644 --- a/base/applications/mspaint/canvas.h +++ b/base/applications/mspaint/canvas.h @@ -21,11 +21,12 @@ public: MESSAGE_HANDLER(WM_MOUSEWHEEL, OnMouseWheel) END_MSG_MAP() + VOID Update(HWND hwndFrom); + +protected: LRESULT OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); }; - -void UpdateCanvas(HWND hwndFrom); diff --git a/base/applications/mspaint/imgarea.cpp b/base/applications/mspaint/imgarea.cpp index 22264f9bdce..1d8b05024ce 100644 --- a/base/applications/mspaint/imgarea.cpp +++ b/base/applications/mspaint/imgarea.cpp @@ -54,7 +54,7 @@ LRESULT CImgAreaWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bH { if (!IsWindow()) return 0; - UpdateCanvas(NULL); + canvasWindow.Update(NULL); return 0; } @@ -346,7 +346,7 @@ LRESULT CImgAreaWindow::OnMouseLeave(UINT nMsg, WPARAM wParam, LPARAM lParam, BO LRESULT CImgAreaWindow::OnImageModelDimensionsChanged(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - UpdateCanvas(NULL); + canvasWindow.Update(NULL); return 0; }

1 0

[reactos] 01/01: [MSPAINT] Rename scrollbox as canvas
by Katayama Hirofumi MZ 16 Mar '23

16 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=7361592ede2b7b554e6d7… commit 7361592ede2b7b554e6d7b4c6680c3a8d612d644 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 07:45:14 2023 +0900 Commit: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> CommitDate: Fri Mar 17 07:45:14 2023 +0900 [MSPAINT] Rename scrollbox as canvas - s/CScrollboxWindow/CCanvasWindow/ - s/scrollboxWindow/canvasWindow/ - s/UpdateScrollbox/UpdateCanvas/ CORE-18867 --- base/applications/mspaint/CMakeLists.txt | 2 +- .../mspaint/{scrollbox.cpp => canvas.cpp} | 38 +++++++++++----------- .../applications/mspaint/{scrollbox.h => canvas.h} | 12 +++---- base/applications/mspaint/globalvar.h | 4 +-- base/applications/mspaint/imgarea.cpp | 12 +++---- base/applications/mspaint/main.cpp | 28 ++++++++-------- base/applications/mspaint/precomp.h | 2 +- base/applications/mspaint/winproc.cpp | 20 ++++++------ 8 files changed, 59 insertions(+), 59 deletions(-) diff --git a/base/applications/mspaint/CMakeLists.txt b/base/applications/mspaint/CMakeLists.txt index b4613246f45..2d6e24dc46d 100644 --- a/base/applications/mspaint/CMakeLists.txt +++ b/base/applications/mspaint/CMakeLists.txt @@ -7,6 +7,7 @@ if(DBG) endif() list(APPEND SOURCE + canvas.cpp dialogs.cpp dib.cpp drawing.cpp @@ -19,7 +20,6 @@ list(APPEND SOURCE palette.cpp palettemodel.cpp registry.cpp - scrollbox.cpp selection.cpp selectionmodel.cpp sizebox.cpp diff --git a/base/applications/mspaint/scrollbox.cpp b/base/applications/mspaint/canvas.cpp similarity index 79% rename from base/applications/mspaint/scrollbox.cpp rename to base/applications/mspaint/canvas.cpp index 2d36711dcc4..1293471ddbf 100644 --- a/base/applications/mspaint/scrollbox.cpp +++ b/base/applications/mspaint/canvas.cpp @@ -1,8 +1,8 @@ /* * PROJECT: PAINT for ReactOS * LICENSE: LGPL - * FILE: base/applications/mspaint/scrollbox.cpp - * PURPOSE: Functionality surrounding the scroll box window class + * FILE: base/applications/mspaint/canvas.cpp + * PURPOSE: Providing the canvas window class * PROGRAMMERS: Benedikt Freisen */ @@ -14,22 +14,22 @@ /* FUNCTIONS ********************************************************/ void -UpdateScrollbox(HWND hwndFrom) +UpdateCanvas(HWND hwndFrom) { CRect tempRect; - scrollboxWindow.GetClientRect(&tempRect); + canvasWindow.GetClientRect(&tempRect); CSize sizeScrollBox(tempRect.Width(), tempRect.Height()); CSize sizeZoomed = { Zoomed(imageModel.GetWidth()), Zoomed(imageModel.GetHeight()) }; CSize sizeWhole = { sizeZoomed.cx + (GRIP_SIZE * 2), sizeZoomed.cy + (GRIP_SIZE * 2) }; /* show/hide the scrollbars */ - scrollboxWindow.ShowScrollBar(SB_HORZ, sizeScrollBox.cx < sizeWhole.cx); - scrollboxWindow.ShowScrollBar(SB_VERT, sizeScrollBox.cy < sizeWhole.cy); + canvasWindow.ShowScrollBar(SB_HORZ, sizeScrollBox.cx < sizeWhole.cx); + canvasWindow.ShowScrollBar(SB_VERT, sizeScrollBox.cy < sizeWhole.cy); if (sizeScrollBox.cx < sizeWhole.cx || sizeScrollBox.cy < sizeWhole.cy) { - scrollboxWindow.GetClientRect(&tempRect); + canvasWindow.GetClientRect(&tempRect); sizeScrollBox = CSize(tempRect.Width(), tempRect.Height()); } @@ -38,14 +38,14 @@ UpdateScrollbox(HWND hwndFrom) si.nMax = sizeWhole.cx; si.nPage = sizeScrollBox.cx; - scrollboxWindow.SetScrollInfo(SB_HORZ, &si); + canvasWindow.SetScrollInfo(SB_HORZ, &si); si.nMax = sizeWhole.cy; si.nPage = sizeScrollBox.cy; - scrollboxWindow.SetScrollInfo(SB_VERT, &si); + canvasWindow.SetScrollInfo(SB_VERT, &si); - INT dx = -scrollboxWindow.GetScrollPos(SB_HORZ); - INT dy = -scrollboxWindow.GetScrollPos(SB_VERT); + INT dx = -canvasWindow.GetScrollPos(SB_HORZ); + INT dy = -canvasWindow.GetScrollPos(SB_VERT); if (sizeboxLeftTop.IsWindow()) { @@ -77,16 +77,16 @@ UpdateScrollbox(HWND hwndFrom) imageArea.MoveWindow(dx + GRIP_SIZE, dy + GRIP_SIZE, sizeZoomed.cx, sizeZoomed.cy, TRUE); } -LRESULT CScrollboxWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +LRESULT CCanvasWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { if (m_hWnd) { - UpdateScrollbox(m_hWnd); + UpdateCanvas(m_hWnd); } return 0; } -LRESULT CScrollboxWindow::OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +LRESULT CCanvasWindow::OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { SCROLLINFO si; si.cbSize = sizeof(SCROLLINFO); @@ -112,11 +112,11 @@ LRESULT CScrollboxWindow::OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOO break; } SetScrollInfo(SB_HORZ, &si); - UpdateScrollbox(m_hWnd); + UpdateCanvas(m_hWnd); return 0; } -LRESULT CScrollboxWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +LRESULT CCanvasWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { SCROLLINFO si; si.cbSize = sizeof(SCROLLINFO); @@ -142,11 +142,11 @@ LRESULT CScrollboxWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOO break; } SetScrollInfo(SB_VERT, &si); - UpdateScrollbox(m_hWnd); + UpdateCanvas(m_hWnd); return 0; } -LRESULT CScrollboxWindow::OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +LRESULT CCanvasWindow::OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { switch (toolsModel.GetActiveTool()) { @@ -167,7 +167,7 @@ LRESULT CScrollboxWindow::OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, return 0; } -LRESULT CScrollboxWindow::OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) +LRESULT CCanvasWindow::OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { return ::SendMessage(GetParent(), nMsg, wParam, lParam); } diff --git a/base/applications/mspaint/scrollbox.h b/base/applications/mspaint/canvas.h similarity index 70% rename from base/applications/mspaint/scrollbox.h rename to base/applications/mspaint/canvas.h index 56822089bf6..b97d43a497d 100644 --- a/base/applications/mspaint/scrollbox.h +++ b/base/applications/mspaint/canvas.h @@ -1,19 +1,19 @@ /* * PROJECT: PAINT for ReactOS * LICENSE: LGPL - * FILE: base/applications/mspaint/scrollbox.h - * PURPOSE: Functionality surrounding the scroll box window class + * FILE: base/applications/mspaint/canvas.h + * PURPOSE: Providing the canvas window class * PROGRAMMERS: Benedikt Freisen */ #pragma once -class CScrollboxWindow : public CWindowImpl<CScrollboxWindow> +class CCanvasWindow : public CWindowImpl<CCanvasWindow> { public: - DECLARE_WND_CLASS_EX(_T("Scrollbox"), 0, COLOR_APPWORKSPACE) + DECLARE_WND_CLASS_EX(_T("ReactOSPaintCanvas"), 0, COLOR_APPWORKSPACE) - BEGIN_MSG_MAP(CScrollboxWindow) + BEGIN_MSG_MAP(CCanvasWindow) MESSAGE_HANDLER(WM_SIZE, OnSize) MESSAGE_HANDLER(WM_HSCROLL, OnHScroll) MESSAGE_HANDLER(WM_VSCROLL, OnVScroll) @@ -28,4 +28,4 @@ public: LRESULT OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); }; -void UpdateScrollbox(HWND hwndFrom); +void UpdateCanvas(HWND hwndFrom); diff --git a/base/applications/mspaint/globalvar.h b/base/applications/mspaint/globalvar.h index 6ffcfa0ea50..aa144275968 100644 --- a/base/applications/mspaint/globalvar.h +++ b/base/applications/mspaint/globalvar.h @@ -61,7 +61,7 @@ class CMiniatureWindow; class CToolBox; class CToolSettingsWindow; class CPaletteWindow; -class CScrollboxWindow; +class CCanvasWindow; class CSelectionWindow; class CImgAreaWindow; class CSizeboxWindow; @@ -73,7 +73,7 @@ extern CMiniatureWindow miniature; extern CToolBox toolBoxContainer; extern CToolSettingsWindow toolSettingsWindow; extern CPaletteWindow paletteWindow; -extern CScrollboxWindow scrollboxWindow; +extern CCanvasWindow canvasWindow; extern CSelectionWindow selectionWindow; extern CImgAreaWindow imageArea; extern CSizeboxWindow sizeboxLeftTop; diff --git a/base/applications/mspaint/imgarea.cpp b/base/applications/mspaint/imgarea.cpp index 5f646da7411..22264f9bdce 100644 --- a/base/applications/mspaint/imgarea.cpp +++ b/base/applications/mspaint/imgarea.cpp @@ -22,10 +22,10 @@ void CImgAreaWindow::drawZoomFrame(int mouseX, int mouseY) LOGBRUSH logbrush; int rop; - RECT clientRectScrollbox; + RECT clientRectCanvas; RECT clientRectImageArea; int x, y, w, h; - scrollboxWindow.GetClientRect(&clientRectScrollbox); + canvasWindow.GetClientRect(&clientRectCanvas); GetClientRect(&clientRectImageArea); w = clientRectImageArea.right * 2; h = clientRectImageArea.bottom * 2; @@ -33,8 +33,8 @@ void CImgAreaWindow::drawZoomFrame(int mouseX, int mouseY) { return; } - w = clientRectImageArea.right * clientRectScrollbox.right / w; - h = clientRectImageArea.bottom * clientRectScrollbox.bottom / h; + w = clientRectImageArea.right * clientRectCanvas.right / w; + h = clientRectImageArea.bottom * clientRectCanvas.bottom / h; x = max(0, min(clientRectImageArea.right - w, mouseX - w / 2)); y = max(0, min(clientRectImageArea.bottom - h, mouseY - h / 2)); @@ -54,7 +54,7 @@ LRESULT CImgAreaWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bH { if (!IsWindow()) return 0; - UpdateScrollbox(NULL); + UpdateCanvas(NULL); return 0; } @@ -346,7 +346,7 @@ LRESULT CImgAreaWindow::OnMouseLeave(UINT nMsg, WPARAM wParam, LPARAM lParam, BO LRESULT CImgAreaWindow::OnImageModelDimensionsChanged(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - UpdateScrollbox(NULL); + UpdateCanvas(NULL); return 0; } diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index 525a4634b95..75cc5862110 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -60,7 +60,7 @@ CMiniatureWindow miniature; CToolBox toolBoxContainer; CToolSettingsWindow toolSettingsWindow; CPaletteWindow paletteWindow; -CScrollboxWindow scrollboxWindow; +CCanvasWindow canvasWindow; CSelectionWindow selectionWindow; CImgAreaWindow imageArea; CSizeboxWindow sizeboxLeftTop; @@ -223,9 +223,9 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument RECT paletteWindowPos = {56, 9, 56 + 255, 9 + 32}; paletteWindow.Create(hwnd, paletteWindowPos, NULL, WS_CHILD | WS_VISIBLE); - /* creating the scroll box */ - RECT scrollboxWindowPos = {0, 0, 0 + 500, 0 + 500}; - scrollboxWindow.Create(hwnd, scrollboxWindowPos, NULL, + // creating the canvas + RECT canvasWindowPos = {0, 0, 0 + 500, 0 + 500}; + canvasWindow.Create(hwnd, canvasWindowPos, NULL, WS_CHILD | WS_GROUP | WS_HSCROLL | WS_VSCROLL | WS_VISIBLE, WS_EX_CLIENTEDGE); /* creating the status bar */ @@ -236,9 +236,9 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument if (registrySettings.ShowStatusBar) ShowWindow(hStatusBar, SW_SHOWNOACTIVATE); - // Creating the window inside the scroll box + // Creating the window inside the canvas RECT imageAreaPos = {GRIP_SIZE, GRIP_SIZE, GRIP_SIZE + imageModel.GetWidth(), GRIP_SIZE + imageModel.GetHeight()}; - imageArea.Create(scrollboxWindow.m_hWnd, imageAreaPos, NULL, WS_CHILD | WS_VISIBLE); + imageArea.Create(canvasWindow.m_hWnd, imageAreaPos, NULL, WS_CHILD | WS_VISIBLE); /* create selection window (initially hidden) */ RECT selectionWindowPos = {350, 0, 350 + 100, 0 + 100}; @@ -308,14 +308,14 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument /* creating the size boxes */ RECT sizeboxPos = {0, 0, GRIP_SIZE, GRIP_SIZE}; - sizeboxLeftTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxCenterTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxLeftCenter.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightCenter.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxLeftBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxCenterBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftTop.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxCenterTop.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightTop.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftCenter.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightCenter.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftBottom.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxCenterBottom.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightBottom.Create(canvasWindow, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); /* placing the size boxes around the image */ imageArea.SendMessage(WM_SIZE, 0, 0); diff --git a/base/applications/mspaint/precomp.h b/base/applications/mspaint/precomp.h index 25fd0329d36..7c1f23acebb 100644 --- a/base/applications/mspaint/precomp.h +++ b/base/applications/mspaint/precomp.h @@ -39,6 +39,7 @@ #define WM_SELECTIONMODELREFRESHNEEDED (WM_APP + 7) #include "resource.h" +#include "canvas.h" #include "drawing.h" #include "dib.h" #include "fullscreen.h" @@ -49,7 +50,6 @@ #include "palette.h" #include "palettemodel.h" #include "registry.h" -#include "scrollbox.h" #include "selection.h" #include "selectionmodel.h" #include "sizebox.h" diff --git a/base/applications/mspaint/winproc.cpp b/base/applications/mspaint/winproc.cpp index e6669e19e68..471853bad20 100644 --- a/base/applications/mspaint/winproc.cpp +++ b/base/applications/mspaint/winproc.cpp @@ -46,7 +46,7 @@ zoomTo(int newZoom, int mouseX, int mouseY) RECT clientRectScrollbox; RECT clientRectImageArea; int x, y, w, h; - scrollboxWindow.GetClientRect(&clientRectScrollbox); + canvasWindow.GetClientRect(&clientRectScrollbox); imageArea.GetClientRect(&clientRectImageArea); w = clientRectImageArea.right * newZoom / toolsModel.GetZoom(); h = clientRectImageArea.bottom * newZoom / toolsModel.GetZoom(); @@ -62,11 +62,11 @@ zoomTo(int newZoom, int mouseX, int mouseY) toolsModel.SetZoom(newZoom); imageArea.MoveWindow(GRIP_SIZE, GRIP_SIZE, Zoomed(imageModel.GetWidth()), Zoomed(imageModel.GetHeight()), FALSE); - scrollboxWindow.Invalidate(TRUE); + canvasWindow.Invalidate(TRUE); imageArea.Invalidate(FALSE); - scrollboxWindow.SendMessage(WM_HSCROLL, MAKEWPARAM(SB_THUMBPOSITION, x), 0); - scrollboxWindow.SendMessage(WM_VSCROLL, MAKEWPARAM(SB_THUMBPOSITION, y), 0); + canvasWindow.SendMessage(WM_HSCROLL, MAKEWPARAM(SB_THUMBPOSITION, x), 0); + canvasWindow.SendMessage(WM_VSCROLL, MAKEWPARAM(SB_THUMBPOSITION, y), 0); return TRUE; } @@ -102,9 +102,9 @@ void CMainWindow::alignChildrenToMainWindow() rcSpace.top += CY_PALETTE; } - if (scrollboxWindow.IsWindow()) + if (canvasWindow.IsWindow()) { - hDWP = ::DeferWindowPos(hDWP, scrollboxWindow, NULL, + hDWP = ::DeferWindowPos(hDWP, canvasWindow, NULL, rcSpace.left, rcSpace.top, rcSpace.right - rcSpace.left, rcSpace.bottom - rcSpace.top, SWP_NOZORDER | SWP_NOACTIVATE | SWP_NOREPOSITION); @@ -216,9 +216,9 @@ LRESULT CMainWindow::OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& for (UINT i = 0; i < nCount; ++i) { if (zDelta < 0) - ::PostMessageW(scrollboxWindow, WM_HSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0); + ::PostMessageW(canvasWindow, WM_HSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0); else if (zDelta > 0) - ::PostMessageW(scrollboxWindow, WM_HSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0); + ::PostMessageW(canvasWindow, WM_HSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0); } } else @@ -227,9 +227,9 @@ LRESULT CMainWindow::OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& for (UINT i = 0; i < nCount; ++i) { if (zDelta < 0) - ::PostMessageW(scrollboxWindow, WM_VSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0); + ::PostMessageW(canvasWindow, WM_VSCROLL, MAKEWPARAM(SB_LINEDOWN, 0), 0); else if (zDelta > 0) - ::PostMessageW(scrollboxWindow, WM_VSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0); + ::PostMessageW(canvasWindow, WM_VSCROLL, MAKEWPARAM(SB_LINEUP, 0), 0); } } }

1 0

[reactos] 01/01: [MSPAINT] Remove extra scrlClientWindow (#5162)
by Katayama Hirofumi MZ 16 Mar '23

16 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=a88dcbd6c3b2d611fb915… commit a88dcbd6c3b2d611fb915025f990c7fc9104f354 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 07:28:42 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Fri Mar 17 07:28:42 2023 +0900 [MSPAINT] Remove extra scrlClientWindow (#5162) CORE-18867 --- base/applications/mspaint/globalvar.h | 1 - base/applications/mspaint/imgarea.cpp | 50 +------- base/applications/mspaint/main.cpp | 30 ++--- base/applications/mspaint/scrollbox.cpp | 219 ++++++++++++++------------------ base/applications/mspaint/scrollbox.h | 2 +- base/applications/mspaint/selection.cpp | 2 - 6 files changed, 110 insertions(+), 194 deletions(-) diff --git a/base/applications/mspaint/globalvar.h b/base/applications/mspaint/globalvar.h index de01287c4da..6ffcfa0ea50 100644 --- a/base/applications/mspaint/globalvar.h +++ b/base/applications/mspaint/globalvar.h @@ -74,7 +74,6 @@ extern CToolBox toolBoxContainer; extern CToolSettingsWindow toolSettingsWindow; extern CPaletteWindow paletteWindow; extern CScrollboxWindow scrollboxWindow; -extern CScrollboxWindow scrlClientWindow; extern CSelectionWindow selectionWindow; extern CImgAreaWindow imageArea; extern CSizeboxWindow sizeboxLeftTop; diff --git a/base/applications/mspaint/imgarea.cpp b/base/applications/mspaint/imgarea.cpp index c0a5f832251..5f646da7411 100644 --- a/base/applications/mspaint/imgarea.cpp +++ b/base/applications/mspaint/imgarea.cpp @@ -14,22 +14,6 @@ /* FUNCTIONS ********************************************************/ -void -updateCanvasAndScrollbars() -{ - selectionWindow.ShowWindow(SW_HIDE); - - int zoomedWidth = Zoomed(imageModel.GetWidth()); - int zoomedHeight = Zoomed(imageModel.GetHeight()); - imageArea.MoveWindow(GRIP_SIZE, GRIP_SIZE, zoomedWidth, zoomedHeight, FALSE); - - scrollboxWindow.Invalidate(TRUE); - imageArea.Invalidate(FALSE); - - scrollboxWindow.SetScrollPos(SB_HORZ, 0, TRUE); - scrollboxWindow.SetScrollPos(SB_VERT, 0, TRUE); -} - void CImgAreaWindow::drawZoomFrame(int mouseX, int mouseY) { HDC hdc; @@ -68,37 +52,9 @@ void CImgAreaWindow::drawZoomFrame(int mouseX, int mouseY) LRESULT CImgAreaWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - if (!IsWindow() || !sizeboxLeftTop.IsWindow()) + if (!IsWindow()) return 0; - int imgXRes = imageModel.GetWidth(); - int imgYRes = imageModel.GetHeight(); - sizeboxLeftTop.MoveWindow( - 0, - 0, GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxCenterTop.MoveWindow( - GRIP_SIZE + (Zoomed(imgXRes) - GRIP_SIZE) / 2, - 0, GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxRightTop.MoveWindow( - GRIP_SIZE + Zoomed(imgXRes), - 0, GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxLeftCenter.MoveWindow( - 0, - GRIP_SIZE + (Zoomed(imgYRes) - GRIP_SIZE) / 2, - GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxRightCenter.MoveWindow( - GRIP_SIZE + Zoomed(imgXRes), - GRIP_SIZE + (Zoomed(imgYRes) - GRIP_SIZE) / 2, - GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxLeftBottom.MoveWindow( - 0, - GRIP_SIZE + Zoomed(imgYRes), GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxCenterBottom.MoveWindow( - GRIP_SIZE + (Zoomed(imgXRes) - GRIP_SIZE) / 2, - GRIP_SIZE + Zoomed(imgYRes), GRIP_SIZE, GRIP_SIZE, TRUE); - sizeboxRightBottom.MoveWindow( - GRIP_SIZE + Zoomed(imgXRes), - GRIP_SIZE + Zoomed(imgYRes), GRIP_SIZE, GRIP_SIZE, TRUE); - UpdateScrollbox(); + UpdateScrollbox(NULL); return 0; } @@ -390,7 +346,7 @@ LRESULT CImgAreaWindow::OnMouseLeave(UINT nMsg, WPARAM wParam, LPARAM lParam, BO LRESULT CImgAreaWindow::OnImageModelDimensionsChanged(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - updateCanvasAndScrollbars(); + UpdateScrollbox(NULL); return 0; } diff --git a/base/applications/mspaint/main.cpp b/base/applications/mspaint/main.cpp index 5ee34cd3240..525a4634b95 100644 --- a/base/applications/mspaint/main.cpp +++ b/base/applications/mspaint/main.cpp @@ -61,7 +61,6 @@ CToolBox toolBoxContainer; CToolSettingsWindow toolSettingsWindow; CPaletteWindow paletteWindow; CScrollboxWindow scrollboxWindow; -CScrollboxWindow scrlClientWindow; CSelectionWindow selectionWindow; CImgAreaWindow imageArea; CSizeboxWindow sizeboxLeftTop; @@ -225,7 +224,7 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument paletteWindow.Create(hwnd, paletteWindowPos, NULL, WS_CHILD | WS_VISIBLE); /* creating the scroll box */ - RECT scrollboxWindowPos = {56, 49, 56 + 472, 49 + 248}; + RECT scrollboxWindowPos = {0, 0, 0 + 500, 0 + 500}; scrollboxWindow.Create(hwnd, scrollboxWindowPos, NULL, WS_CHILD | WS_GROUP | WS_HSCROLL | WS_VSCROLL | WS_VISIBLE, WS_EX_CLIENTEDGE); @@ -237,16 +236,13 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument if (registrySettings.ShowStatusBar) ShowWindow(hStatusBar, SW_SHOWNOACTIVATE); - RECT scrlClientWindowPos = {0, 0, 0 + 500, 0 + 500}; - scrlClientWindow.Create(scrollboxWindow.m_hWnd, scrlClientWindowPos, NULL, WS_CHILD | WS_VISIBLE); + // Creating the window inside the scroll box + RECT imageAreaPos = {GRIP_SIZE, GRIP_SIZE, GRIP_SIZE + imageModel.GetWidth(), GRIP_SIZE + imageModel.GetHeight()}; + imageArea.Create(scrollboxWindow.m_hWnd, imageAreaPos, NULL, WS_CHILD | WS_VISIBLE); /* create selection window (initially hidden) */ RECT selectionWindowPos = {350, 0, 350 + 100, 0 + 100}; - selectionWindow.Create(scrlClientWindow.m_hWnd, selectionWindowPos, NULL, WS_CHILD | BS_OWNERDRAW); - - /* creating the window inside the scroll box, on which the image in hDrawingDC's bitmap is drawn */ - RECT imageAreaPos = {GRIP_SIZE, GRIP_SIZE, GRIP_SIZE + imageModel.GetWidth(), GRIP_SIZE + imageModel.GetHeight()}; - imageArea.Create(scrlClientWindow.m_hWnd, imageAreaPos, NULL, WS_CHILD | WS_VISIBLE); + selectionWindow.Create(imageArea.m_hWnd, selectionWindowPos, NULL, WS_CHILD | BS_OWNERDRAW); if (__argc >= 2) { @@ -312,14 +308,14 @@ _tWinMain (HINSTANCE hThisInstance, HINSTANCE hPrevInstance, LPTSTR lpszArgument /* creating the size boxes */ RECT sizeboxPos = {0, 0, GRIP_SIZE, GRIP_SIZE}; - sizeboxLeftTop.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxCenterTop.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightTop.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxLeftCenter.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightCenter.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxLeftBottom.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxCenterBottom.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); - sizeboxRightBottom.Create(scrlClientWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxCenterTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightTop.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftCenter.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightCenter.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxLeftBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxCenterBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); + sizeboxRightBottom.Create(scrollboxWindow.m_hWnd, sizeboxPos, NULL, WS_CHILD | WS_VISIBLE); /* placing the size boxes around the image */ imageArea.SendMessage(WM_SIZE, 0, 0); diff --git a/base/applications/mspaint/scrollbox.cpp b/base/applications/mspaint/scrollbox.cpp index c05dffd5ec9..2d36711dcc4 100644 --- a/base/applications/mspaint/scrollbox.cpp +++ b/base/applications/mspaint/scrollbox.cpp @@ -11,176 +11,143 @@ #include "precomp.h" #include <atltypes.h> -/* - * Scrollbar functional modes: - * 0 view < canvas - * 1 view < canvas + scroll width - * 2 view >= canvas + scroll width - * - * Matrix of scrollbar presence (VERTICAL,HORIZONTAL) given by - * vertical & horizontal scrollbar modes (view:canvas ratio): - * - * horizontal mode - * | 0 | 1 | 2 - * vertical ---+-------------+-------------+------------ - * mode 0 | TRUE,TRUE | TRUE,TRUE | TRUE,FALSE - * ---+-------------+-------------+------------ - * 1 | TRUE,TRUE | FALSE,FALSE | FALSE,FALSE - * ---+-------------+-------------+------------ - * 2 | FALSE,TRUE | FALSE,FALSE | FALSE,FALSE - */ - -struct ScrollbarPresence -{ - BOOL bVert; - BOOL bHoriz; -}; - -CONST ScrollbarPresence sp_mx[3][3] = -{ - { { TRUE,TRUE }, { TRUE,TRUE }, { TRUE,FALSE } }, - { { TRUE,TRUE }, { FALSE,FALSE }, { FALSE,FALSE } }, - { { FALSE,TRUE }, { FALSE,FALSE }, { FALSE,FALSE } } -}; - -CONST INT HSCROLL_WIDTH = ::GetSystemMetrics(SM_CYHSCROLL); -CONST INT VSCROLL_WIDTH = ::GetSystemMetrics(SM_CXVSCROLL); - - /* FUNCTIONS ********************************************************/ void -UpdateScrollbox() +UpdateScrollbox(HWND hwndFrom) { CRect tempRect; - CSize sizeImageArea; - CSize sizeScrollBox; - INT vmode, hmode; - SCROLLINFO si; + scrollboxWindow.GetClientRect(&tempRect); + CSize sizeScrollBox(tempRect.Width(), tempRect.Height()); - scrollboxWindow.GetWindowRect(&tempRect); - sizeScrollBox = CSize(tempRect.Width(), tempRect.Height()); - - if (imageArea.IsWindow()) - imageArea.GetClientRect(&tempRect); - sizeImageArea = CSize(tempRect.Width(), tempRect.Height()); - sizeImageArea += CSize(GRIP_SIZE * 2, GRIP_SIZE * 2); + CSize sizeZoomed = { Zoomed(imageModel.GetWidth()), Zoomed(imageModel.GetHeight()) }; + CSize sizeWhole = { sizeZoomed.cx + (GRIP_SIZE * 2), sizeZoomed.cy + (GRIP_SIZE * 2) }; /* show/hide the scrollbars */ - vmode = (sizeScrollBox.cy < sizeImageArea.cy ? 0 : - (sizeScrollBox.cy < sizeImageArea.cy + HSCROLL_WIDTH ? 1 : 2)); - hmode = (sizeScrollBox.cx < sizeImageArea.cx ? 0 : - (sizeScrollBox.cx < sizeImageArea.cx + VSCROLL_WIDTH ? 1 : 2)); - scrollboxWindow.ShowScrollBar(SB_VERT, sp_mx[vmode][hmode].bVert); - scrollboxWindow.ShowScrollBar(SB_HORZ, sp_mx[vmode][hmode].bHoriz); + scrollboxWindow.ShowScrollBar(SB_HORZ, sizeScrollBox.cx < sizeWhole.cx); + scrollboxWindow.ShowScrollBar(SB_VERT, sizeScrollBox.cy < sizeWhole.cy); - si.cbSize = sizeof(SCROLLINFO); - si.fMask = SIF_PAGE | SIF_RANGE; + if (sizeScrollBox.cx < sizeWhole.cx || sizeScrollBox.cy < sizeWhole.cy) + { + scrollboxWindow.GetClientRect(&tempRect); + sizeScrollBox = CSize(tempRect.Width(), tempRect.Height()); + } + + SCROLLINFO si = { sizeof(si), SIF_PAGE | SIF_RANGE }; si.nMin = 0; - si.nMax = sizeImageArea.cx + - (sp_mx[vmode][hmode].bVert == TRUE ? HSCROLL_WIDTH : 0); + si.nMax = sizeWhole.cx; si.nPage = sizeScrollBox.cx; scrollboxWindow.SetScrollInfo(SB_HORZ, &si); - si.nMax = sizeImageArea.cy + - (sp_mx[vmode][hmode].bHoriz == TRUE ? VSCROLL_WIDTH : 0); + si.nMax = sizeWhole.cy; si.nPage = sizeScrollBox.cy; scrollboxWindow.SetScrollInfo(SB_VERT, &si); - if (scrlClientWindow.IsWindow()) + INT dx = -scrollboxWindow.GetScrollPos(SB_HORZ); + INT dy = -scrollboxWindow.GetScrollPos(SB_VERT); + + if (sizeboxLeftTop.IsWindow()) { - scrlClientWindow.MoveWindow( - -scrollboxWindow.GetScrollPos(SB_HORZ), -scrollboxWindow.GetScrollPos(SB_VERT), - sizeImageArea.cx, sizeImageArea.cy, TRUE); + sizeboxLeftTop.MoveWindow(dx, dy, GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxCenterTop.MoveWindow(dx + GRIP_SIZE + (sizeZoomed.cx - GRIP_SIZE) / 2, + dy, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxRightTop.MoveWindow(dx + GRIP_SIZE + sizeZoomed.cx, + dy, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxLeftCenter.MoveWindow(dx, + dy + GRIP_SIZE + (sizeZoomed.cy - GRIP_SIZE) / 2, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxRightCenter.MoveWindow(dx + GRIP_SIZE + sizeZoomed.cx, + dy + GRIP_SIZE + (sizeZoomed.cy - GRIP_SIZE) / 2, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxLeftBottom.MoveWindow(dx, + dy + GRIP_SIZE + sizeZoomed.cy, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxCenterBottom.MoveWindow(dx + GRIP_SIZE + (sizeZoomed.cx - GRIP_SIZE) / 2, + dy + GRIP_SIZE + sizeZoomed.cy, + GRIP_SIZE, GRIP_SIZE, TRUE); + sizeboxRightBottom.MoveWindow(dx + GRIP_SIZE + sizeZoomed.cx, + dy + GRIP_SIZE + sizeZoomed.cy, + GRIP_SIZE, GRIP_SIZE, TRUE); } + + if (imageArea.IsWindow() && hwndFrom != imageArea.m_hWnd) + imageArea.MoveWindow(dx + GRIP_SIZE, dy + GRIP_SIZE, sizeZoomed.cx, sizeZoomed.cy, TRUE); } LRESULT CScrollboxWindow::OnSize(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - if (m_hWnd && m_hWnd == scrollboxWindow.m_hWnd) + if (m_hWnd) { - UpdateScrollbox(); + UpdateScrollbox(m_hWnd); } return 0; } LRESULT CScrollboxWindow::OnHScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - if (m_hWnd == scrollboxWindow.m_hWnd) + SCROLLINFO si; + si.cbSize = sizeof(SCROLLINFO); + si.fMask = SIF_ALL; + GetScrollInfo(SB_HORZ, &si); + switch (LOWORD(wParam)) { - SCROLLINFO si; - si.cbSize = sizeof(SCROLLINFO); - si.fMask = SIF_ALL; - scrollboxWindow.GetScrollInfo(SB_HORZ, &si); - switch (LOWORD(wParam)) - { - case SB_THUMBTRACK: - case SB_THUMBPOSITION: - si.nPos = HIWORD(wParam); - break; - case SB_LINELEFT: - si.nPos -= 5; - break; - case SB_LINERIGHT: - si.nPos += 5; - break; - case SB_PAGELEFT: - si.nPos -= si.nPage; - break; - case SB_PAGERIGHT: - si.nPos += si.nPage; - break; - } - scrollboxWindow.SetScrollInfo(SB_HORZ, &si); - scrlClientWindow.MoveWindow(-scrollboxWindow.GetScrollPos(SB_HORZ), - -scrollboxWindow.GetScrollPos(SB_VERT), - Zoomed(imageModel.GetWidth()) + 2 * GRIP_SIZE, - Zoomed(imageModel.GetHeight()) + 2 * GRIP_SIZE, TRUE); + case SB_THUMBTRACK: + case SB_THUMBPOSITION: + si.nPos = HIWORD(wParam); + break; + case SB_LINELEFT: + si.nPos -= 5; + break; + case SB_LINERIGHT: + si.nPos += 5; + break; + case SB_PAGELEFT: + si.nPos -= si.nPage; + break; + case SB_PAGERIGHT: + si.nPos += si.nPage; + break; } + SetScrollInfo(SB_HORZ, &si); + UpdateScrollbox(m_hWnd); return 0; } LRESULT CScrollboxWindow::OnVScroll(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - if (m_hWnd == scrollboxWindow.m_hWnd) + SCROLLINFO si; + si.cbSize = sizeof(SCROLLINFO); + si.fMask = SIF_ALL; + GetScrollInfo(SB_VERT, &si); + switch (LOWORD(wParam)) { - SCROLLINFO si; - si.cbSize = sizeof(SCROLLINFO); - si.fMask = SIF_ALL; - scrollboxWindow.GetScrollInfo(SB_VERT, &si); - switch (LOWORD(wParam)) - { - case SB_THUMBTRACK: - case SB_THUMBPOSITION: - si.nPos = HIWORD(wParam); - break; - case SB_LINEUP: - si.nPos -= 5; - break; - case SB_LINEDOWN: - si.nPos += 5; - break; - case SB_PAGEUP: - si.nPos -= si.nPage; - break; - case SB_PAGEDOWN: - si.nPos += si.nPage; - break; - } - scrollboxWindow.SetScrollInfo(SB_VERT, &si); - scrlClientWindow.MoveWindow(-scrollboxWindow.GetScrollPos(SB_HORZ), - -scrollboxWindow.GetScrollPos(SB_VERT), - Zoomed(imageModel.GetWidth()) + 2 * GRIP_SIZE, - Zoomed(imageModel.GetHeight()) + 2 * GRIP_SIZE, TRUE); + case SB_THUMBTRACK: + case SB_THUMBPOSITION: + si.nPos = HIWORD(wParam); + break; + case SB_LINEUP: + si.nPos -= 5; + break; + case SB_LINEDOWN: + si.nPos += 5; + break; + case SB_PAGEUP: + si.nPos -= si.nPage; + break; + case SB_PAGEDOWN: + si.nPos += si.nPage; + break; } + SetScrollInfo(SB_VERT, &si); + UpdateScrollbox(m_hWnd); return 0; } LRESULT CScrollboxWindow::OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { - selectionWindow.ShowWindow(SW_HIDE); - switch (toolsModel.GetActiveTool()) { case TOOL_BEZIER: diff --git a/base/applications/mspaint/scrollbox.h b/base/applications/mspaint/scrollbox.h index e2fa3461847..56822089bf6 100644 --- a/base/applications/mspaint/scrollbox.h +++ b/base/applications/mspaint/scrollbox.h @@ -28,4 +28,4 @@ public: LRESULT OnMouseWheel(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); }; -void UpdateScrollbox(); +void UpdateScrollbox(HWND hwndFrom); diff --git a/base/applications/mspaint/selection.cpp b/base/applications/mspaint/selection.cpp index 3a87a21f19d..64d01538216 100644 --- a/base/applications/mspaint/selection.cpp +++ b/base/applications/mspaint/selection.cpp @@ -107,8 +107,6 @@ LRESULT CSelectionWindow::OnLButtonDown(UINT nMsg, WPARAM wParam, LPARAM lParam, if (m_iAction != ACTION_MOVE) SetCursor(LoadCursor(NULL, m_lpszCursorLUT[m_iAction])); m_bMoving = TRUE; - scrlClientWindow.InvalidateRect(NULL, TRUE); - scrlClientWindow.SendMessage(WM_PAINT, 0, 0); imageArea.InvalidateRect(NULL, FALSE); imageArea.SendMessage(WM_PAINT, 0, 0); m_rgbBack = paletteModel.GetBgColor();

1 0

[reactos] 01/01: [MSPAINT] Refactor about Recent Files (#5163)
by Katayama Hirofumi MZ 16 Mar '23

16 Mar '23

https://git.reactos.org/?p=reactos.git;a=commitdiff;h=41c30182d4638d270b493… commit 41c30182d4638d270b493dee631946aaec5d8b51 Author: Katayama Hirofumi MZ <katayama.hirofumi.mz(a)gmail.com> AuthorDate: Fri Mar 17 07:28:01 2023 +0900 Commit: GitHub <noreply(a)github.com> CommitDate: Fri Mar 17 07:28:01 2023 +0900 [MSPAINT] Refactor about Recent Files (#5163) Define MAX_RECENT_FILES macro as 4. Remove strFile1, ..., strFile4 settings and add strFiles[MAX_RECENT_FILES] for Most Recently Used (MRU) files. CORE-18867 --- base/applications/mspaint/registry.cpp | 71 ++++++++++------------- base/applications/mspaint/registry.h | 7 +-- base/applications/mspaint/winproc.cpp | 103 +++++++++++++++++---------------- base/applications/mspaint/winproc.h | 2 + 4 files changed, 88 insertions(+), 95 deletions(-) diff --git a/base/applications/mspaint/registry.cpp b/base/applications/mspaint/registry.cpp index c027fcac0e7..46119989429 100644 --- a/base/applications/mspaint/registry.cpp +++ b/base/applications/mspaint/registry.cpp @@ -115,10 +115,12 @@ void RegistrySettings::Load(INT nCmdShow) CRegKey files; if (files.Open(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Windows\\CurrentVersion\\Applets\\Paint\\Recent File List"), KEY_READ) == ERROR_SUCCESS) { - ReadString(files, _T("File1"), strFile1); - ReadString(files, _T("File2"), strFile2); - ReadString(files, _T("File3"), strFile3); - ReadString(files, _T("File4"), strFile4); + TCHAR szName[64]; + for (INT i = 0; i < MAX_RECENT_FILES; ++i) + { + wsprintf(szName, _T("File%u"), i + 1); + ReadString(files, szName, strFiles[i]); + } } CRegKey text; @@ -167,14 +169,12 @@ void RegistrySettings::Store() CRegKey files; if (files.Create(HKEY_CURRENT_USER, _T("Software\\Microsoft\\Windows\\CurrentVersion\\Applets\\Paint\\Recent File List")) == ERROR_SUCCESS) { - if (!strFile1.IsEmpty()) - files.SetStringValue(_T("File1"), strFile1); - if (!strFile2.IsEmpty()) - files.SetStringValue(_T("File2"), strFile2); - if (!strFile3.IsEmpty()) - files.SetStringValue(_T("File3"), strFile3); - if (!strFile4.IsEmpty()) - files.SetStringValue(_T("File4"), strFile4); + TCHAR szName[64]; + for (INT iFile = 0; iFile < MAX_RECENT_FILES; ++iFile) + { + wsprintf(szName, _T("File%u"), iFile + 1); + files.SetStringValue(szName, strFiles[iFile]); + } } CRegKey text; @@ -194,39 +194,30 @@ void RegistrySettings::Store() void RegistrySettings::SetMostRecentFile(LPCTSTR szPathName) { + // Register the file to the user's 'Recent' folder if (szPathName && szPathName[0]) SHAddToRecentDocs(SHARD_PATHW, szPathName); - if (strFile1 == szPathName) + // If szPathName is present in strFiles, move it to the top of the list + for (INT i = MAX_RECENT_FILES - 1, iFound = -1; i > 0; --i) { - // do nothing + if (iFound < 0 && strFiles[i].CompareNoCase(szPathName) == 0) + iFound = i; + + if (iFound >= 0) + { + CString tmp = strFiles[i]; + strFiles[i] = strFiles[i - 1]; + strFiles[i - 1] = tmp; + } } - else if (strFile2 == szPathName) - { - CString strTemp = strFile2; - strFile2 = strFile1; - strFile1 = strTemp; - } - else if (strFile3 == szPathName) - { - CString strTemp = strFile3; - strFile3 = strFile2; - strFile2 = strFile1; - strFile1 = strTemp; - } - else if (strFile4 == szPathName) - { - CString strTemp = strFile4; - strFile4 = strFile3; - strFile3 = strFile2; - strFile2 = strFile1; - strFile1 = strTemp; - } - else + + // If szPathName is not the first item in strFiles, insert it at the top of the list + if (strFiles[0].CompareNoCase(szPathName) != 0) { - strFile4 = strFile3; - strFile3 = strFile2; - strFile2 = strFile1; - strFile1 = szPathName; + for (INT i = MAX_RECENT_FILES - 1; i > 0; --i) + strFiles[i] = strFiles[i - 1]; + + strFiles[0] = szPathName; } } diff --git a/base/applications/mspaint/registry.h b/base/applications/mspaint/registry.h index e7b4cc644ee..b891d523c23 100644 --- a/base/applications/mspaint/registry.h +++ b/base/applications/mspaint/registry.h @@ -8,6 +8,8 @@ #pragma once +#define MAX_RECENT_FILES 4 + class RegistrySettings { private: @@ -27,10 +29,7 @@ public: DWORD UnitSetting; WINDOWPLACEMENT WindowPlacement; - CString strFile1; - CString strFile2; - CString strFile3; - CString strFile4; + CString strFiles[MAX_RECENT_FILES]; CString strFontName; DWORD PointSize; diff --git a/base/applications/mspaint/winproc.cpp b/base/applications/mspaint/winproc.cpp index ac96bb2cfcd..e6669e19e68 100644 --- a/base/applications/mspaint/winproc.cpp +++ b/base/applications/mspaint/winproc.cpp @@ -10,6 +10,7 @@ */ #include "precomp.h" +#include <assert.h> typedef HWND (WINAPI *FN_HtmlHelpW)(HWND, LPCWSTR, UINT, DWORD_PTR); @@ -310,53 +311,63 @@ LRESULT CMainWindow::OnClose(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHan return 0; } +void CMainWindow::ProcessFileMenu(HMENU hPopupMenu) +{ + LPCTSTR dotext = PathFindExtensionW(filepathname); + BOOL isBMP = FALSE; + if (_tcsicmp(dotext, _T(".bmp")) == 0 || + _tcsicmp(dotext, _T(".dib")) == 0 || + _tcsicmp(dotext, _T(".rle")) == 0) + { + isBMP = TRUE; + } + + EnableMenuItem(hPopupMenu, IDM_FILEASWALLPAPERPLANE, ENABLED_IF(isAFile && isBMP)); + EnableMenuItem(hPopupMenu, IDM_FILEASWALLPAPERCENTERED, ENABLED_IF(isAFile && isBMP)); + EnableMenuItem(hPopupMenu, IDM_FILEASWALLPAPERSTRETCHED, ENABLED_IF(isAFile && isBMP)); + + for (INT iItem = 0; iItem < MAX_RECENT_FILES; ++iItem) + RemoveMenu(hPopupMenu, IDM_FILE1 + iItem, MF_BYCOMMAND); + + if (registrySettings.strFiles[0].IsEmpty()) + return; + + RemoveMenu(hPopupMenu, IDM_FILEMOSTRECENTLYUSEDFILE, MF_BYCOMMAND); + + INT cMenuItems = GetMenuItemCount(hPopupMenu); + + for (INT iItem = 0; iItem < MAX_RECENT_FILES; ++iItem) + { + CString& strFile = registrySettings.strFiles[iItem]; + if (strFile.IsEmpty()) + break; + + // Condense the lengthy pathname by using '...' +#define MAX_RECENT_PATHNAME_DISPLAY 30 + CPath pathFile(strFile); + pathFile.CompactPathEx(MAX_RECENT_PATHNAME_DISPLAY); + assert(_tcslen((LPCTSTR)pathFile) <= MAX_RECENT_PATHNAME_DISPLAY); + + // Add an accelerator (by '&') to the item number for quick access + TCHAR szText[4 + MAX_RECENT_PATHNAME_DISPLAY + 1]; + wsprintf(szText, _T("&%u %s"), iItem + 1, (LPCTSTR)pathFile); + + INT iMenuItem = (cMenuItems - 2) + iItem; + InsertMenu(hPopupMenu, iMenuItem, MF_BYPOSITION | MF_STRING, IDM_FILE1 + iItem, szText); + } +} + LRESULT CMainWindow::OnInitMenuPopup(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled) { HMENU menu = GetMenu(); BOOL trueSelection = (::IsWindowVisible(selectionWindow) && ((toolsModel.GetActiveTool() == TOOL_FREESEL) || (toolsModel.GetActiveTool() == TOOL_RECTSEL))); - BOOL isBMP; + switch (lParam) { case 0: /* File menu */ - if ((HMENU)wParam != GetSubMenu(menu, 0)) - break; - - isBMP = _wcsicmp(PathFindExtensionW(filepathname), L".bmp") == 0; - EnableMenuItem(menu, IDM_FILEASWALLPAPERPLANE, ENABLED_IF(isAFile && isBMP)); - EnableMenuItem(menu, IDM_FILEASWALLPAPERCENTERED, ENABLED_IF(isAFile && isBMP)); - EnableMenuItem(menu, IDM_FILEASWALLPAPERSTRETCHED, ENABLED_IF(isAFile && isBMP)); - - RemoveMenu(menu, IDM_FILE1, MF_BYCOMMAND); - RemoveMenu(menu, IDM_FILE2, MF_BYCOMMAND); - RemoveMenu(menu, IDM_FILE3, MF_BYCOMMAND); - RemoveMenu(menu, IDM_FILE4, MF_BYCOMMAND); - if (!registrySettings.strFile1.IsEmpty()) - { - RemoveMenu(menu, IDM_FILEMOSTRECENTLYUSEDFILE, MF_BYCOMMAND); - CPath pathFile1(registrySettings.strFile1); - pathFile1.CompactPathEx(30); - if (!registrySettings.strFile2.IsEmpty()) - { - CPath pathFile2(registrySettings.strFile2); - pathFile2.CompactPathEx(30); - if (!registrySettings.strFile3.IsEmpty()) - { - CPath pathFile3(registrySettings.strFile3); - pathFile3.CompactPathEx(30); - if (!registrySettings.strFile4.IsEmpty()) - { - CPath pathFile4(registrySettings.strFile4); - pathFile4.CompactPathEx(30); - InsertMenu((HMENU)wParam, 17, MF_BYPOSITION | MF_STRING, IDM_FILE4, _T("4 ") + pathFile4); - } - InsertMenu((HMENU)wParam, 17, MF_BYPOSITION | MF_STRING, IDM_FILE3, _T("3 ") + pathFile3); - } - InsertMenu((HMENU)wParam, 17, MF_BYPOSITION | MF_STRING, IDM_FILE2, _T("2 ") + pathFile2); - } - InsertMenu((HMENU)wParam, 17, MF_BYPOSITION | MF_STRING, IDM_FILE1, _T("1 ") + pathFile1); - } + ProcessFileMenu((HMENU)wParam); break; case 1: /* Edit menu */ EnableMenuItem(menu, IDM_EDITUNDO, ENABLED_IF(imageModel.HasUndoSteps())); @@ -545,23 +556,13 @@ LRESULT CMainWindow::OnCommand(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bH RegistrySettings::SetWallpaper(filepathname, RegistrySettings::STRETCHED); break; case IDM_FILE1: - { - ConfirmSave() && DoLoadImageFile(m_hWnd, registrySettings.strFile1, TRUE); - break; - } case IDM_FILE2: - { - ConfirmSave() && DoLoadImageFile(m_hWnd, registrySettings.strFile2, TRUE); - break; - } case IDM_FILE3: - { - ConfirmSave() && DoLoadImageFile(m_hWnd, registrySettings.strFile3, TRUE); - break; - } case IDM_FILE4: { - ConfirmSave() && DoLoadImageFile(m_hWnd, registrySettings.strFile4, TRUE); + INT iFile = LOWORD(wParam) - IDM_FILE1; + if (ConfirmSave()) + DoLoadImageFile(m_hWnd, registrySettings.strFiles[iFile], TRUE); break; } case IDM_EDITUNDO: diff --git a/base/applications/mspaint/winproc.h b/base/applications/mspaint/winproc.h index 749277560a3..71db5e7950d 100644 --- a/base/applications/mspaint/winproc.h +++ b/base/applications/mspaint/winproc.h @@ -28,6 +28,7 @@ public: MESSAGE_HANDLER(WM_MOUSEWHEEL, OnMouseWheel) END_MSG_MAP() +private: LRESULT OnDropFiles(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnCreate(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); LRESULT OnDestroy(UINT nMsg, WPARAM wParam, LPARAM lParam, BOOL& bHandled); @@ -44,4 +45,5 @@ public: void saveImage(BOOL overwrite); void InsertSelectionFromHBITMAP(HBITMAP bitmap, HWND window); BOOL ConfirmSave(); + void ProcessFileMenu(HMENU hPopupMenu); };

1 0

Jump to page:

2026

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

2007

2006

2005

2004

Ros-diffs