From ffc00b42e9eff7b8bd022ff6c129f083d5437445 Mon Sep 17 00:00:00 2001 From: Lioncash Date: Thu, 13 Nov 2014 20:10:36 -0500 Subject: [PATCH] MemoryView: Fix a dealloc of a stack reference stack allocation only works with the root menu. Any other children must be heap allocated. These however, will be freed by wxWidgets. --- Source/Core/DolphinWX/Debugger/MemoryView.cpp | 10 +++++----- 1 file changed, 5 insertions(+), 5 deletions(-) diff --git a/Source/Core/DolphinWX/Debugger/MemoryView.cpp b/Source/Core/DolphinWX/Debugger/MemoryView.cpp index 86b0215f2a..4f87ef7763 100644 --- a/Source/Core/DolphinWX/Debugger/MemoryView.cpp +++ b/Source/Core/DolphinWX/Debugger/MemoryView.cpp @@ -230,11 +230,11 @@ void CMemoryView::OnMouseDownR(wxMouseEvent& event) menu.Append(IDM_WATCHADDRESS, _("Add to &watch")); menu.Append(IDM_TOGGLEMEMORY, _("Toggle &memory")); - wxMenu viewAsSubMenu; - viewAsSubMenu.Append(IDM_VIEWASFP, _("FP value")); - viewAsSubMenu.Append(IDM_VIEWASASCII, "ASCII"); - viewAsSubMenu.Append(IDM_VIEWASHEX, _("Hex")); - menu.AppendSubMenu(&viewAsSubMenu, _("View As:")); + wxMenu* viewAsSubMenu = new wxMenu; + viewAsSubMenu->Append(IDM_VIEWASFP, _("FP value")); + viewAsSubMenu->Append(IDM_VIEWASASCII, "ASCII"); + viewAsSubMenu->Append(IDM_VIEWASHEX, _("Hex")); + menu.AppendSubMenu(viewAsSubMenu, _("View As:")); PopupMenu(&menu); }