winaccessibility/inc/AccTopWindowListener.hxx | 6 +++--- winaccessibility/source/service/msaaservice_impl.cxx | 5 ++--- 2 files changed, 5 insertions(+), 6 deletions(-)
New commits: commit 2daa13389bc491792c8e5a29c97208f851d526ae Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Thu Dec 5 09:57:31 2024 +0000 Commit: Michael Weghorn <m.wegh...@posteo.de> CommitDate: Fri Dec 6 22:37:18 2024 +0100 wina11y: Drop unnecessary cast to sal_uInt16 Application::GetTopWindow also takes a tools::Long param, so there's no reason to cast to sal_uInt16. Change-Id: Ic592262bee8173329986ccb83298a26deb3586b2 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177879 Reviewed-by: Michael Weghorn <m.wegh...@posteo.de> Tested-by: Jenkins (cherry picked from commit 5ce6b81d4dc599ed4470f164b221eeefd07f7fba) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177944 diff --git a/winaccessibility/source/service/msaaservice_impl.cxx b/winaccessibility/source/service/msaaservice_impl.cxx index 3891b6df4872..e858d2b367be 100644 --- a/winaccessibility/source/service/msaaservice_impl.cxx +++ b/winaccessibility/source/service/msaaservice_impl.cxx @@ -210,9 +210,8 @@ static void AccessBridgeHandleExistingWindow(const Reference< XMSAAService > &xA */ static void AccessBridgeUpdateOldTopWindows( const Reference< XMSAAService > &xAccMgr ) { - sal_uInt16 nTopWindowCount = static_cast<sal_uInt16>(Application::GetTopWindowCount()); - - for ( sal_uInt16 i = 0; i < nTopWindowCount; i++ ) + tools::Long nTopWindowCount = Application::GetTopWindowCount(); + for (tools::Long i = 0; i < nTopWindowCount; i++) { vcl::Window* pTopWindow = Application::GetTopWindow( i ); css::uno::Reference< css::accessibility::XAccessible > xAccessible = pTopWindow->GetAccessible(); commit c6c35c2950507d0dd4c6db7a4b39916fe13ca1e2 Author: Michael Weghorn <m.wegh...@posteo.de> AuthorDate: Thu Dec 5 09:39:56 2024 +0000 Commit: Michael Weghorn <m.wegh...@posteo.de> CommitDate: Fri Dec 6 22:37:06 2024 +0100 wina11y: Make these 2 methods non-virtual Change-Id: I791ec6d822573e18c981343d3ce8371acf0b3b38 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177878 Tested-by: Jenkins Reviewed-by: Michael Weghorn <m.wegh...@posteo.de> (cherry picked from commit d1c40cfda530cc249ba1daa0bbb8dc71ffbc2d46) Reviewed-on: https://gerrit.libreoffice.org/c/core/+/177943 diff --git a/winaccessibility/inc/AccTopWindowListener.hxx b/winaccessibility/inc/AccTopWindowListener.hxx index d48ad035dd6a..9d8b46f95acc 100644 --- a/winaccessibility/inc/AccTopWindowListener.hxx +++ b/winaccessibility/inc/AccTopWindowListener.hxx @@ -55,10 +55,10 @@ public: // XEventListener virtual void SAL_CALL disposing(const css::lang::EventObject& Source) override; - virtual void AddAllListeners(css::accessibility::XAccessible* pAccessible, - css::accessibility::XAccessible* pParentXAcc, HWND pWND); + void AddAllListeners(css::accessibility::XAccessible* pAccessible, + css::accessibility::XAccessible* pParentXAcc, HWND pWND); //for On-Demand load. - virtual void HandleWindowOpened(css::accessibility::XAccessible* pAccessible); + void HandleWindowOpened(css::accessibility::XAccessible* pAccessible); sal_Int64 GetMSComPtr(sal_Int64 hWnd, sal_Int64 lParam, sal_Int64 wParam); };