X-Git-Url: https://git.saurik.com/wxWidgets.git/blobdiff_plain/48580976469f7c46b730f96effacfa680e55e3cd..503271a5743fb372ad7cfce3438a11e3aca782fc:/src/cocoa/menu.mm diff --git a/src/cocoa/menu.mm b/src/cocoa/menu.mm index 2d662e4111..529f422350 100644 --- a/src/cocoa/menu.mm +++ b/src/cocoa/menu.mm @@ -6,7 +6,7 @@ // Created: 2002/12/09 // RCS-ID: $Id: // Copyright: (c) 2002 David Elliott -// Licence: wxWindows license +// Licence: wxWidgets licence ///////////////////////////////////////////////////////////////////////////// // ============================================================================ @@ -45,12 +45,15 @@ bool wxMenu::Create(const wxString& title, long style) { wxAutoNSAutoreleasePool pool; m_cocoaNSMenu = [[NSMenu alloc] initWithTitle: wxNSStringWithWxString(title)]; + AssociateNSMenu(m_cocoaNSMenu); return true; } wxMenu::~wxMenu() { - [m_cocoaNSMenu release]; + DisassociateNSMenu(m_cocoaNSMenu); + if(!m_cocoaDeletes) + [m_cocoaNSMenu release]; } wxMenuItem* wxMenu::DoAppend(wxMenuItem *item) @@ -80,6 +83,33 @@ wxMenuItem* wxMenu::DoRemove(wxMenuItem *item) return retitem; } +// This autoreleases the menu on the assumption that something is +// going to retain it shortly (for instance, it is going to be returned from +// an overloaded [NSStatusItem menu] or from the applicationDockMenu: +// NSApplication delegate method. +// +// It then sets a bool flag m_cocoaDeletes. When the NSMenu is dealloc'd +// (dealloc is the Cocoa destructor) we delete ourselves. In this manner we +// can be available for Cocoa calls until Cocoa is finished with us. +// +// I can see very few reasons to undo this. Nevertheless, it is implemented. +void wxMenu::SetCocoaDeletes(bool cocoaDeletes) +{ + if(m_cocoaDeletes==cocoaDeletes) + return; + m_cocoaDeletes = cocoaDeletes; + if(m_cocoaDeletes) + [m_cocoaNSMenu autorelease]; + else + [m_cocoaNSMenu retain]; +} + +void wxMenu::Cocoa_dealloc() +{ + if(m_cocoaDeletes) + delete this; +} + // ============================================================================ // wxMenuBar implementation // ============================================================================