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);
 };

Reply via email to