[Libreoffice-commits] core.git: 2 commits - winaccessibility/inc winaccessibility/source
Michael Weghorn (via logerrit)
logerrit at kemper.freedesktop.org
Mon Sep 20 16:05:24 UTC 2021
winaccessibility/inc/AccObject.hxx | 4 +---
winaccessibility/source/UAccCOM/MAccessible.cxx | 7 +++----
winaccessibility/source/UAccCOM/MAccessible.h | 2 +-
winaccessibility/source/UAccCOMIDL/UAccCOM.idl | 2 +-
winaccessibility/source/service/AccObject.cxx | 10 ++--------
winaccessibility/source/service/AccObjectWinManager.cxx | 2 +-
6 files changed, 9 insertions(+), 18 deletions(-)
New commits:
commit 85b4b15bb02f8d2fdbead0c836763621096b995e
Author: Michael Weghorn <m.weghorn at posteo.de>
AuthorDate: Mon Sep 20 09:23:10 2021 +0100
Commit: Michael Weghorn <m.weghorn at posteo.de>
CommitDate: Mon Sep 20 18:05:00 2021 +0200
wina11y: Drop bool param from CMAccessible::NotifyDestroy
... and 'AccObject::NotifyDestroy', respectively.
The only calling site calls it with 'true' as parameter
(and 'CMAccessible::NotifyDestroy' was calling
'm_xAccessible.clear()' regardless of what value the
passed param had anyway).
Change-Id: I08a24f79b50ec327ae461bd44ee6460a2d8145f7
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122343
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn at posteo.de>
diff --git a/winaccessibility/inc/AccObject.hxx b/winaccessibility/inc/AccObject.hxx
index bc689178c95c..2a246ce43b29 100644
--- a/winaccessibility/inc/AccObject.hxx
+++ b/winaccessibility/inc/AccObject.hxx
@@ -107,7 +107,7 @@ public:
void DeleteChild( AccObject* pChild );
AccObject* NextChild();
- void NotifyDestroy(bool bDestroy);
+ void NotifyDestroy();
void DecreaseState(short xState );//call COM interface DecreaseState method
void IncreaseState( short xState );//call COM interface IncreaseState method
diff --git a/winaccessibility/source/UAccCOM/MAccessible.cxx b/winaccessibility/source/UAccCOM/MAccessible.cxx
index 2d62ec494bc7..8f69dcf01649 100644
--- a/winaccessibility/source/UAccCOM/MAccessible.cxx
+++ b/winaccessibility/source/UAccCOM/MAccessible.cxx
@@ -1364,15 +1364,14 @@ COM_DECLSPEC_NOTHROW STDMETHODIMP CMAccessible::Put_XAccAgent(hyper pAgent)
/**
* When a UNO control disposing, it disposes its listeners,
* then notify AccObject in bridge management, then notify
-* COM that the XAccessible is invalid,so set m_xAccessible as NULL
-* @param isDestroy, true is it needs to be destroyed.
+* COM that the XAccessible is invalid, so set m_xAccessible as NULL
* @return S_OK if successful and E_FAIL if failure.
*/
-COM_DECLSPEC_NOTHROW STDMETHODIMP CMAccessible::NotifyDestroy(BOOL isDestroy)
+COM_DECLSPEC_NOTHROW STDMETHODIMP CMAccessible::NotifyDestroy()
{
// internal IMAccessible - no mutex meeded
- m_isDestroy = isDestroy;
+ m_isDestroy = true;
m_xAccessible.clear();
return S_OK;
}
diff --git a/winaccessibility/source/UAccCOM/MAccessible.h b/winaccessibility/source/UAccCOM/MAccessible.h
index 71296cfc2fda..224e59179d61 100644
--- a/winaccessibility/source/UAccCOM/MAccessible.h
+++ b/winaccessibility/source/UAccCOM/MAccessible.h
@@ -147,7 +147,7 @@ public:
STDMETHOD(Put_XAccWindowHandle)(HWND hwnd) override;
STDMETHOD(Put_XAccChildID)(long dChildID) override;
STDMETHOD(Put_XAccAgent)(hyper pAgent) override;
- STDMETHOD(NotifyDestroy)(BOOL isDestroy) override;
+ STDMETHOD(NotifyDestroy)() override;
STDMETHOD(Put_ActionDescription)( const OLECHAR* szAction) override;
STDMETHOD(SetDefaultAction)(hyper pAction) override;
STDMETHOD(GetUNOInterface)(hyper*) override;
diff --git a/winaccessibility/source/UAccCOMIDL/UAccCOM.idl b/winaccessibility/source/UAccCOMIDL/UAccCOM.idl
index 1a0705754114..da123ab7f972 100644
--- a/winaccessibility/source/UAccCOMIDL/UAccCOM.idl
+++ b/winaccessibility/source/UAccCOMIDL/UAccCOM.idl
@@ -49,7 +49,7 @@ import "defines.idl";
[id(23), helpstring("method SetDefaultAction")] HRESULT SetDefaultAction(hyper pAction);
[id(24), helpstring("method Put_ActionDescription")] HRESULT Put_ActionDescription( const OLECHAR* szAction);
[id(25), helpstring("method Put_XAccAgent")] HRESULT Put_XAccAgent(hyper pAgent);
- [id(26), helpstring("method NotifyDestroy")] HRESULT NotifyDestroy(BOOL isDestroy);
+ [id(26), helpstring("method NotifyDestroy")] HRESULT NotifyDestroy();
[id(30), helpstring("method Get_XAccChildID")] HRESULT Get_XAccChildID([out,retval] long* childID);
};
[
diff --git a/winaccessibility/source/service/AccObject.cxx b/winaccessibility/source/service/AccObject.cxx
index 7d1adb27afd4..877d33cc0e37 100644
--- a/winaccessibility/source/service/AccObject.cxx
+++ b/winaccessibility/source/service/AccObject.cxx
@@ -1116,10 +1116,10 @@ void AccObject::GetExpandedState( sal_Bool* isExpandable, sal_Bool* isExpanded)
}
}
-void AccObject::NotifyDestroy(bool bDestroy)
+void AccObject::NotifyDestroy()
{
if(m_pIMAcc)
- m_pIMAcc->NotifyDestroy(bDestroy);
+ m_pIMAcc->NotifyDestroy();
}
void AccObject::SetParentObj(AccObject* pParentAccObj)
diff --git a/winaccessibility/source/service/AccObjectWinManager.cxx b/winaccessibility/source/service/AccObjectWinManager.cxx
index 39f9f4a621f9..89d3e7db5d5e 100644
--- a/winaccessibility/source/service/AccObjectWinManager.cxx
+++ b/winaccessibility/source/service/AccObjectWinManager.cxx
@@ -1154,7 +1154,7 @@ void AccObjectWinManager::NotifyDestroy(XAccessible* pXAcc)
AccObject* accObj = GetAccObjByXAcc(pXAcc);
if(accObj)
{
- accObj->NotifyDestroy(true);
+ accObj->NotifyDestroy();
}
}
commit 185b183a71924c84edb385ec5eb4fe6d4bfe8e05
Author: Michael Weghorn <m.weghorn at posteo.de>
AuthorDate: Mon Sep 20 08:40:05 2021 +0100
Commit: Michael Weghorn <m.weghorn at posteo.de>
CommitDate: Mon Sep 20 18:04:49 2021 +0200
wina11y: Drop AccObject::m_bShouldDestroy
The getter, 'AccObject::ifShouldDestroy',
was unused, so drop it.
The only place where the 'm_bShouldDestroy' member
was actually used was in 'AccObject::NotifyDestroy',
where it was assigned a value from a method param that
was then passed further right away, so there's no need to
store the value at all.
Change-Id: Id5688e9742892e345b143a8eebeed5099696a0f2
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/122342
Tested-by: Jenkins
Reviewed-by: Michael Weghorn <m.weghorn at posteo.de>
diff --git a/winaccessibility/inc/AccObject.hxx b/winaccessibility/inc/AccObject.hxx
index 2b2232323a02..bc689178c95c 100644
--- a/winaccessibility/inc/AccObject.hxx
+++ b/winaccessibility/inc/AccObject.hxx
@@ -59,7 +59,6 @@ private:
short m_accRole;
long m_resID;
HWND m_pParantID;
- bool m_bShouldDestroy; //avoid access COM interface when acc object is deleted
IMAccessible* m_pIMAcc;
AccObject* m_pParentObj;
IAccChildList m_childrenList;
@@ -108,8 +107,7 @@ public:
void DeleteChild( AccObject* pChild );
AccObject* NextChild();
- void NotifyDestroy(bool ifDelete);
- bool ifShouldDestroy();
+ void NotifyDestroy(bool bDestroy);
void DecreaseState(short xState );//call COM interface DecreaseState method
void IncreaseState( short xState );//call COM interface IncreaseState method
diff --git a/winaccessibility/source/service/AccObject.cxx b/winaccessibility/source/service/AccObject.cxx
index 9e3f7d063b3b..7d1adb27afd4 100644
--- a/winaccessibility/source/service/AccObject.cxx
+++ b/winaccessibility/source/service/AccObject.cxx
@@ -162,7 +162,6 @@ AccObject::AccObject(XAccessible* pAcc, AccObjectManagerAgent* pAgent,
AccEventListener* pListener) :
m_resID (NULL),
m_pParantID (nullptr),
- m_bShouldDestroy(false),
m_pIMAcc (nullptr),
m_pParentObj(nullptr),
m_pListener (pListener),
@@ -1117,11 +1116,10 @@ void AccObject::GetExpandedState( sal_Bool* isExpandable, sal_Bool* isExpanded)
}
}
-void AccObject::NotifyDestroy(bool ifDelete)
+void AccObject::NotifyDestroy(bool bDestroy)
{
- m_bShouldDestroy=ifDelete;
if(m_pIMAcc)
- m_pIMAcc->NotifyDestroy(m_bShouldDestroy);
+ m_pIMAcc->NotifyDestroy(bDestroy);
}
void AccObject::SetParentObj(AccObject* pParentAccObj)
@@ -1187,9 +1185,5 @@ AccObject* AccObject::GetParentObj()
{
return m_pParentObj;
}
-bool AccObject::ifShouldDestroy()
-{
- return m_bShouldDestroy;
-}
/* vim:set shiftwidth=4 softtabstop=4 expandtab: */
More information about the Libreoffice-commits
mailing list