diff options
| author | Méven Car <[email protected]> | 2026-06-13 13:50:08 +0000 |
|---|---|---|
| committer | Méven Car <[email protected]> | 2026-06-13 13:50:08 +0000 |
| commit | cc806587766d09f916de712b0b5e248ba0c5383c (patch) | |
| tree | a8b30e2eda43bec6f113b092991513d1df94c481 /src/itemactions/setfoldericonitemaction.cpp | |
| parent | 70f34211cb4ad1f3f6170ded062bafff85d66643 (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/setfoldericonitemaction.cpp')
| -rw-r--r-- | src/itemactions/setfoldericonitemaction.cpp | 7 |
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; |
