diff options
author | Stephan Bergmann <sbergman@redhat.com> | 2023-11-08 13:54:14 +0100 |
---|---|---|
committer | Stephan Bergmann <sbergman@redhat.com> | 2023-11-08 18:57:36 +0100 |
commit | 0a2622760a967e9d64cfdc632548dd42c1836324 (patch) | |
tree | a19651e3637e5b996e629150b37e783779dd8ad0 /winaccessibility | |
parent | 94dbf63dcd1843b4d7d0515d63998adb8032ac3c (diff) |
Drop presumably redundant null check
In a18bdb3bc05e761704cc345a66a9d642bc4f4a0a "Integrate branch of IAccessible2",
this had originally been
> if (pAgent && pAgent->IsStateManageDescendant(pXAccessible))
> {
> return;
> }
so lets assume the `pAgent` check was only there to avoid a null deref
Change-Id: I320c181ff7d86d7c328bf9e5139dce2df1facb7d
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/159130
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
Diffstat (limited to 'winaccessibility')
-rw-r--r-- | winaccessibility/source/service/AccContainerEventListener.cxx | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/winaccessibility/source/service/AccContainerEventListener.cxx b/winaccessibility/source/service/AccContainerEventListener.cxx index f0736edf35d2..d05b9f5a27e1 100644 --- a/winaccessibility/source/service/AccContainerEventListener.cxx +++ b/winaccessibility/source/service/AccContainerEventListener.cxx @@ -438,7 +438,7 @@ void AccContainerEventListener::UpdateAllChildrenState(XAccessible* pXAccessible return; } - if (m_pObjManager && AccObjectWinManager::IsStateManageDescendant(pXAccessible)) + if (AccObjectWinManager::IsStateManageDescendant(pXAccessible)) { return; } |