From bb2da89e49ca7588b1da9cf49885cf01a7c05bb9 Mon Sep 17 00:00:00 2001 From: Sebastian Englbrecht Date: Tue, 26 May 2026 20:49:15 +0200 Subject: DolphinRecentTabsMenu: remove no-op null assignment after delete action is a by-value parameter; action = nullptr only affects the local copy. Remove the dead line. Found with cppcheck --- src/dolphinrecenttabsmenu.cpp | 1 - 1 file changed, 1 deletion(-) (limited to 'src/dolphinrecenttabsmenu.cpp') diff --git a/src/dolphinrecenttabsmenu.cpp b/src/dolphinrecenttabsmenu.cpp index d9b103c0c..ce39f3f53 100644 --- a/src/dolphinrecenttabsmenu.cpp +++ b/src/dolphinrecenttabsmenu.cpp @@ -79,7 +79,6 @@ void DolphinRecentTabsMenu::handleAction(QAction *action) const QByteArray state = action->data().toByteArray(); removeAction(action); delete action; - action = nullptr; Q_EMIT restoreClosedTab(state); Q_EMIT closedTabsCountChanged(menu()->actions().size() - 2); } -- cgit v1.3.1