diff options
| author | Michael Weghorn <m.weghorn@posteo.de> | 2025-01-28 08:49:19 +0100 |
|---|---|---|
| committer | Michael Weghorn <m.weghorn@posteo.de> | 2025-01-29 08:25:01 +0100 |
| commit | 0267279ae8c2145cfedc1497634a869fb79e300e (patch) | |
| tree | 38ef7ac277692fc1570843a92b2983bd5efed876 | |
| parent | f3271cf5adfbe2e00d02417d85ff320763d7e9a5 (diff) | |
uno grid a11y: Drop superfluous check
The non-null case already has an early return
above, so there's no need to check again whether
`m_pAccessibleTable` is actually nullptr.
Change-Id: Id094c80787f425fe18e48cf0a6460c50d33056ef
Reviewed-on: https://gerrit.libreoffice.org/c/core/+/180816
Reviewed-by: Michael Weghorn <m.weghorn@posteo.de>
Tested-by: Jenkins
| -rw-r--r-- | toolkit/source/controls/table/tablecontrol_impl.cxx | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/toolkit/source/controls/table/tablecontrol_impl.cxx b/toolkit/source/controls/table/tablecontrol_impl.cxx index ac04b86a2255..f6dc5c161f5f 100644 --- a/toolkit/source/controls/table/tablecontrol_impl.cxx +++ b/toolkit/source/controls/table/tablecontrol_impl.cxx @@ -2356,15 +2356,13 @@ namespace svt::table return m_pAccessibleTable; DBG_TESTSOLARMUTEX(); - if ( m_pAccessibleTable == nullptr ) + + Reference<XAccessible> const xAccParent = i_parentWindow.GetAccessible(); + if (xAccParent.is()) { - Reference< XAccessible > const xAccParent = i_parentWindow.GetAccessible(); - if ( xAccParent.is() ) - { - m_pAccessibleTable = new accessibility::AccessibleGridControlAccess( - xAccParent, m_rAntiImpl - ); - } + m_pAccessibleTable = new accessibility::AccessibleGridControlAccess( + xAccParent, m_rAntiImpl + ); } return m_pAccessibleTable; |