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

[Michael Weghorn (via logerrit)]

 svx/source/table/accessiblecell.cxx |    1 -
 1 file changed, 1 deletion(-)

New commits:
commit 8acbe35ee38419bfa484c8c43e810c8ac68201c7
Author:     Michael Weghorn <m.wegh...@posteo.de>
AuthorDate: Tue Aug 30 15:04:09 2022 +0200
Commit:     Michael Weghorn <m.wegh...@posteo.de>
CommitDate: Tue Aug 30 23:49:09 2022 +0200

    svx a11y: Drop TODO that doesn't need action
    
    `AccessibleTextHelper_Impl::getAccessibleChild` that
    gets called from `AccessibleTextHelper::GetChild`
    throws an `IndexOutOfBoundsException` on invalid
    index which is the expected behavior here and what
    the comment for `AccessibleCell::getAccessibleChild`
    says, so drop the TODO that seems to have been there
    to check this.
    
    Change-Id: If04438cc3625ce438ada75bb6dea7e6c88c3e894
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/139062
    Tested-by: Jenkins
    Reviewed-by: Michael Weghorn <m.wegh...@posteo.de>

diff --git a/svx/source/table/accessiblecell.cxx 
b/svx/source/table/accessiblecell.cxx
index 84dbe792cf08..1d6321084b22 100644
--- a/svx/source/table/accessiblecell.cxx
+++ b/svx/source/table/accessiblecell.cxx
@@ -167,7 +167,6 @@ Reference<XAccessible> SAL_CALL 
AccessibleCell::getAccessibleChild (sal_Int32 nI
     SolarMutexGuard aSolarGuard;
     ThrowIfDisposed ();
 
-    // todo: does GetChild throw IndexOutOfBoundsException?
     return mpText->GetChild (nIndex);
 }