┌   ┐
54
└   ┘

summaryrefslogtreecommitdiff
path: root/src/itemactions
diff options
context:
space:
mode:
authorMéven Car <[email protected]>2026-06-13 13:50:08 +0000
committerMéven Car <[email protected]>2026-06-13 13:50:08 +0000
commitcc806587766d09f916de712b0b5e248ba0c5383c (patch)
treea8b30e2eda43bec6f113b092991513d1df94c481 /src/itemactions
parent70f34211cb4ad1f3f6170ded062bafff85d66643 (diff)
setfoldericon: fix leak of the submenu
ButtonsWithSubMenuWidgetAction owned the QMenu (no parent, setMenu does not take ownership) but had no destructor, so every folder context menu leaked one QMenu and its overflow actions. Add a destructor that deletes it.
Diffstat (limited to 'src/itemactions')
-rw-r--r--src/itemactions/setfoldericonitemaction.cpp7
1 files changed, 7 insertions, 0 deletions
diff --git a/src/itemactions/setfoldericonitemaction.cpp b/src/itemactions/setfoldericonitemaction.cpp
index feb6adc2b..a90864d86 100644
--- a/src/itemactions/setfoldericonitemaction.cpp
+++ b/src/itemactions/setfoldericonitemaction.cpp
@@ -78,6 +78,13 @@ public:
{
}
+ ~ButtonsWithSubMenuWidgetAction() override
+ {
+ // m_subMenu has no parent (QPushButton::setMenu does not take ownership),
+ // so this action is its sole owner.
+ delete m_subMenu;
+ }
+
void setActions(const QList<QAction *> &actions)
{
m_actions = actions;