From e0b17563b8f61b47b67eab7e5b2fc3cee8fe72b7 Mon Sep 17 00:00:00 2001
From: Volker Hilsheimer <volker.hilsheimer@qt.io>
Date: Tue, 18 Mar 2025 15:03:26 +0100
Subject: [PATCH] macOS a11y: collapse branches on identical condition
MIME-Version: 1.0
Content-Type: text/plain; charset=UTF-8
Content-Transfer-Encoding: 8bit

No need to test twice whether the row's columns array is initialized.

Pick-to: 6.8
Change-Id: Iead06f98ad421202bb4b71ae8efbd784ab3e87c1
Reviewed-by: Tor Arne Vestbø <tor.arne.vestbo@qt.io>
(cherry picked from commit bd6e55c856e3c016048410f69919e443c71628bf)
Reviewed-by: Qt Cherry-pick Bot <cherrypick_bot@qt-project.org>
---
 src/plugins/platforms/cocoa/qcocoaaccessibilityelement.mm | 2 --
 1 file changed, 2 deletions(-)

diff --git a/src/plugins/platforms/cocoa/qcocoaaccessibilityelement.mm b/src/plugins/platforms/cocoa/qcocoaaccessibilityelement.mm
index a07c9fef3e2..010c34042b5 100644
--- a/src/plugins/platforms/cocoa/qcocoaaccessibilityelement.mm
+++ b/src/plugins/platforms/cocoa/qcocoaaccessibilityelement.mm
@@ -159,8 +159,6 @@ static void convertLineOffset(QAccessibleTextInterface *text, int *line, int *of
                                 qCWarning(lcAccessibilityTable)
                                         << "Table representation column count is out of sync:"
                                         << rowElement->columns.count << "!=" << tableInterface->columnCount();
-                            }
-                            if (rowElement->columns) {
                                 [rowElement->columns autorelease];
                                 rowElement->columns = nil;
                             }