From 2a5da1609d9d39e5b2bd80bd6af80bebcb6c1fb9 Mon Sep 17 00:00:00 2001
From: Michael Weghorn <m.weghorn@posteo.de>
Date: Thu, 16 Sep 2021 14:50:29 +0100
Subject: wina11y: Avoid some casting between HWND and sal_Int64

... by changing the corresponding params for
AccObjectManagerAgent::InsertAccObj and
AccObjectManagerAgent::InsertChildrenAccObj
to HWND.

Change-Id: I598bac30d827ae9166bc21cb5e8dff88a5cc99bf
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122203
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
---
 winaccessibility/source/UAccCOM/MAccessible.cxx           | 3 +--
 winaccessibility/source/service/AccFrameEventListener.cxx | 3 +--
 winaccessibility/source/service/AccObjectManagerAgent.cxx | 9 ++++-----
 winaccessibility/source/service/AccTopWindowListener.cxx  | 3 +--
 4 files changed, 7 insertions(+), 11 deletions(-)

(limited to 'winaccessibility/source')

diff --git a/winaccessibility/source/UAccCOM/MAccessible.cxx b/winaccessibility/source/UAccCOM/MAccessible.cxx
index ed0796c43ec1..325cd00e2a62 100644
--- a/winaccessibility/source/UAccCOM/MAccessible.cxx
+++ b/winaccessibility/source/UAccCOM/MAccessible.cxx
@@ -1415,8 +1415,7 @@ IMAccessible* CMAccessible::GetChildInterface(long dChildID)//for test
 
         if(!isGet)
         {
-            g_pAgent->InsertAccObj(pXChild.get(), m_xAccessible.get(),
-                    reinterpret_cast<sal_Int64>(m_hwnd));
+            g_pAgent->InsertAccObj(pXChild.get(), m_xAccessible.get(), m_hwnd);
             isGet = get_IAccessibleFromXAccessible(pXChild.get(), &pChild);
         }
 
diff --git a/winaccessibility/source/service/AccFrameEventListener.cxx b/winaccessibility/source/service/AccFrameEventListener.cxx
index 2d31bdffb551..4ddaf3d8789b 100644
--- a/winaccessibility/source/service/AccFrameEventListener.cxx
+++ b/winaccessibility/source/service/AccFrameEventListener.cxx
@@ -91,8 +91,7 @@ void AccFrameEventListener::HandleChildChangedEvent(Any oldValue, Any newValue)
                 = pvclwindow->GetWindow()->GetSystemData();
 
             //add this child
-            pAgent->InsertAccObj(pAcc, m_xAccessible.get(),
-                    reinterpret_cast<sal_Int64>(systemdata->hWnd));
+            pAgent->InsertAccObj(pAcc, m_xAccessible.get(), systemdata->hWnd);
             //add all oldValue's existing children
             pAgent->InsertChildrenAccObj(pAcc);
             pAgent->NotifyAccEvent(UM_EVENT_CHILD_ADDED, pAcc);
diff --git a/winaccessibility/source/service/AccObjectManagerAgent.cxx b/winaccessibility/source/service/AccObjectManagerAgent.cxx
index b3290075fb6b..1e0c37729f31 100644
--- a/winaccessibility/source/service/AccObjectManagerAgent.cxx
+++ b/winaccessibility/source/service/AccObjectManagerAgent.cxx
@@ -142,11 +142,10 @@ void  AccObjectManagerAgent::UpdateDescription( XAccessible* pXAcc, Any newDesc
    * @return If the method is correctly processed.
    */
 bool AccObjectManagerAgent::InsertAccObj(
-        XAccessible* pXAcc, XAccessible* pParentXAcc, sal_Int64 nWnd)
+        XAccessible* pXAcc, XAccessible* pParentXAcc, HWND hWnd)
 {
     if( pWinManager )
-        return pWinManager->InsertAccObj(pXAcc, pParentXAcc,
-                static_cast<HWND>(reinterpret_cast<void*>(nWnd)));
+        return pWinManager->InsertAccObj(pXAcc, pParentXAcc, hWnd);
 
     return false;
 }
@@ -174,10 +173,10 @@ AccObjectManagerAgent::SaveTopWindowHandle(sal_Int64 hWnd, XAccessible* pXAcc)
    * @return If the method is correctly processed.
    */
 bool
-AccObjectManagerAgent::InsertChildrenAccObj(XAccessible* pXAcc, sal_Int64 pWnd)
+AccObjectManagerAgent::InsertChildrenAccObj(XAccessible* pXAcc, HWND hWnd)
 {
     if( pWinManager )
-        return pWinManager->InsertChildrenAccObj( pXAcc, reinterpret_cast<HWND>(pWnd) );
+        return pWinManager->InsertChildrenAccObj(pXAcc, hWnd);
 
     return false;
 }
diff --git a/winaccessibility/source/service/AccTopWindowListener.cxx b/winaccessibility/source/service/AccTopWindowListener.cxx
index bcb821771c3c..3c52d3280bd2 100644
--- a/winaccessibility/source/service/AccTopWindowListener.cxx
+++ b/winaccessibility/source/service/AccTopWindowListener.cxx
@@ -139,8 +139,7 @@ void AccTopWindowListener::AddAllListeners(css::accessibility::XAccessible* pAcc
         return;
     }
 
-    accManagerAgent.InsertAccObj(pAccessible, pParentXAcc,
-            reinterpret_cast<sal_Int64>(pWND));
+    accManagerAgent.InsertAccObj(pAccessible, pParentXAcc, pWND);
 
     if (!accManagerAgent.IsContainer(pAccessible))
     {
-- 
cgit