Changeset 99469 in vbox

Timestamp:

Apr 19, 2023 8:35:51 PM (2 years ago)

Author:

vboxsync

svn:sync-xref-src-repo-rev:

156954

Message:

FE/Qt: Don't call the overloaded setShortcuts in ~UIAction, but call the QAction version directly. We only want to make sure the shortcut is removed from QGuiApplication, no need to call updateText() which may then involve already destroyed objects during application (VM) termination.

File:

• trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPool.h (modified) (1 diff)

trunk/src/VBox/Frontends/VirtualBox/src/globals/UIActionPool.h

@@ -237 +237 @@
     UIAction(UIActionPool *pParent, UIActionType enmType, bool fMachineMenuAction = false);
     /** Destructs action. */
-    virtual ~UIAction() RT_OVERRIDE { delete menu(); setShortcuts(QList<QKeySequence>() /*unregister*/); }
+    virtual ~UIAction() RT_OVERRIDE { delete menu(); QAction::setShortcuts(QList<QKeySequence>() /*unregister*/); }
 
     /** Returns action-pool this action belongs to. */