[Libreoffice-commits] core.git: winaccessibility/source

[Stephan Bergmann (via logerrit)]

 winaccessibility/source/service/AccContainerEventListener.cxx |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

New commits:
commit 0a2622760a967e9d64cfdc632548dd42c1836324
Author:     Stephan Bergmann <sberg...@redhat.com>
AuthorDate: Wed Nov 8 13:54:14 2023 +0100
Commit:     Stephan Bergmann <sberg...@redhat.com>
CommitDate: Wed Nov 8 18:57:36 2023 +0100

    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.wegh...@posteo.de>
    Tested-by: Jenkins

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