From 8b51cd0d8a48282f51f4feba45642424c26b56e2 Mon Sep 17 00:00:00 2001 From: ZappoMan Date: Sat, 31 May 2014 14:04:33 -0700 Subject: [PATCH] fix bug in Menu::removeAction() where we weren't removing the action key from the hash, memory leak, and other problems --- interface/src/Menu.cpp | 1 + 1 file changed, 1 insertion(+) diff --git a/interface/src/Menu.cpp b/interface/src/Menu.cpp index 6121e5c458..deb052e595 100644 --- a/interface/src/Menu.cpp +++ b/interface/src/Menu.cpp @@ -858,6 +858,7 @@ QAction* Menu::addCheckableActionToQMenuAndActionHash(QMenu* destinationMenu, void Menu::removeAction(QMenu* menu, const QString& actionName) { menu->removeAction(_actionHash.value(actionName)); + _actionHash.remove(actionName); } void Menu::setIsOptionChecked(const QString& menuOption, bool isChecked) {