coverity#1441483 try and silence bogus Using invalid iterator

Change-Id: I312cef93a0cef48207a3b7f5165dd48d702de3c6
Reviewed-on: https://gerrit.libreoffice.org/63810
Tested-by: Jenkins
Reviewed-by: Caolán McNamara <caolanm@redhat.com>
Tested-by: Caolán McNamara <caolanm@redhat.com>

1 files changed, 4 insertions, 4 deletions

diff --git a/svx/source/accessibility/ChildrenManagerImpl.cxx b/svx/source/accessibility/ChildrenManagerImpl.cxx
index 0485b35d24ab..928438bd6ff2 100644
--- a/svx/source/accessibility/ChildrenManagerImpl.cxx
+++ b/svx/source/accessibility/ChildrenManagerImpl.cxx
@@ -737,8 +737,7 @@ bool ChildrenManagerImpl::ReplaceChild (
     auto I = std::find_if(maVisibleChildren.begin(), maVisibleChildren.end(),
         [&pCurrentChild](const ChildDescriptor& rChild) { return rChild.GetAccessibleShape() == pCurrentChild; });
 
-    bool bResult = I != maVisibleChildren.end();
-    if (bResult)
+    if (I != maVisibleChildren.end())
     {
         // Dispose the current child and send an event about its deletion.
         pCurrentChild->dispose();
@@ -754,12 +753,13 @@ bool ChildrenManagerImpl::ReplaceChild (
             AccessibleEventId::CHILD,
             uno::makeAny (I->mxAccessibleShape),
             uno::Any());
+
+        return true;
     }
 
     // When not found among the visible children we have to search the list
     // of accessible shapes.  This is not yet implemented.
-
-    return bResult;
+    return false;
 }
 
 // Add the impl method for IAccessibleParent interface