Stop using NSApp in the OS X front end.

As of OS X 10.10 (Yosemite), this causes a compile failure, which
according to the Internet is because NSApp is a generic 'id'
(Objective-C's answer to void *), which causes the compiler to select
the wrong method for [NSApp setDelegate:...] and give a spurious type
warning.

Fixed by using [NSApplication sharedApplication] throughout, or
rather, calling that once at startup and assigning the result into my
own global variable which will do as an NSApp replacement.

[originally from svn r10286]
This commit is contained in:
Simon Tatham
2014-10-20 17:43:38 +00:00
parent b1158d4dcf
commit 74a97a1efb

50
osx.m
View File

@ -100,6 +100,12 @@ NSMenu *typemenu;
*/ */
extern const struct drawing_api osx_drawing; extern const struct drawing_api osx_drawing;
/*
* The NSApplication shared instance, which I'll want to refer to from
* a few places here and there.
*/
NSApplication *app;
/* ---------------------------------------------------------------------- /* ----------------------------------------------------------------------
* Miscellaneous support routines that aren't part of any object or * Miscellaneous support routines that aren't part of any object or
* clearly defined subsystem. * clearly defined subsystem.
@ -1191,7 +1197,7 @@ struct frontend {
for (k = 0; k < cfg_ncontrols; k++) for (k = 0; k < cfg_ncontrols; k++)
[[sheet contentView] addSubview:cfg_controls[k]]; [[sheet contentView] addSubview:cfg_controls[k]];
[NSApp beginSheet:sheet modalForWindow:self [app beginSheet:sheet modalForWindow:self
modalDelegate:nil didEndSelector:NULL contextInfo:nil]; modalDelegate:nil didEndSelector:NULL contextInfo:nil];
} }
@ -1213,7 +1219,7 @@ struct frontend {
- (void)sheetEndWithStatus:(BOOL)update - (void)sheetEndWithStatus:(BOOL)update
{ {
assert(sheet != NULL); assert(sheet != NULL);
[NSApp endSheet:sheet]; [app endSheet:sheet];
[sheet orderOut:self]; [sheet orderOut:self];
sheet = NULL; sheet = NULL;
if (update) { if (update) {
@ -1550,7 +1556,7 @@ void activate_timer(frontend *fe)
* AppController: the object which receives the messages from all * AppController: the object which receives the messages from all
* menu selections that aren't standard OS X functions. * menu selections that aren't standard OS X functions.
*/ */
@interface AppController : NSObject @interface AppController : NSObject <NSApplicationDelegate>
{ {
} }
- (void)newGameWindow:(id)sender; - (void)newGameWindow:(id)sender;
@ -1608,27 +1614,27 @@ int main(int argc, char **argv)
pool = [[NSAutoreleasePool alloc] init]; pool = [[NSAutoreleasePool alloc] init];
icon = [NSImage imageNamed:@"NSApplicationIcon"]; icon = [NSImage imageNamed:@"NSApplicationIcon"];
[NSApplication sharedApplication]; app = [NSApplication sharedApplication];
[NSApp setApplicationIconImage:icon]; [app setApplicationIconImage:icon];
controller = [[[AppController alloc] init] autorelease]; controller = [[[AppController alloc] init] autorelease];
[NSApp setDelegate:controller]; [app setDelegate:controller];
[NSApp setMainMenu: newmenu("Main Menu")]; [app setMainMenu: newmenu("Main Menu")];
menu = newsubmenu([NSApp mainMenu], "Apple Menu"); menu = newsubmenu([app mainMenu], "Apple Menu");
newitem(menu, "About Puzzles", "", NULL, @selector(about:)); newitem(menu, "About Puzzles", "", NULL, @selector(about:));
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
[NSApp setServicesMenu:newsubmenu(menu, "Services")]; [app setServicesMenu:newsubmenu(menu, "Services")];
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
newitem(menu, "Hide Puzzles", "h", NSApp, @selector(hide:)); newitem(menu, "Hide Puzzles", "h", app, @selector(hide:));
newitem(menu, "Hide Others", "o-h", NSApp, @selector(hideOtherApplications:)); newitem(menu, "Hide Others", "o-h", app, @selector(hideOtherApplications:));
newitem(menu, "Show All", "", NSApp, @selector(unhideAllApplications:)); newitem(menu, "Show All", "", app, @selector(unhideAllApplications:));
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
newitem(menu, "Quit", "q", NSApp, @selector(terminate:)); newitem(menu, "Quit", "q", app, @selector(terminate:));
[NSApp setAppleMenu: menu]; [app setAppleMenu: menu];
menu = newsubmenu([NSApp mainMenu], "File"); menu = newsubmenu([app mainMenu], "File");
newitem(menu, "Open", "o", NULL, @selector(loadSavedGame:)); newitem(menu, "Open", "o", NULL, @selector(loadSavedGame:));
newitem(menu, "Save As", "s", NULL, @selector(saveGame:)); newitem(menu, "Save As", "s", NULL, @selector(saveGame:));
newitem(menu, "New Game", "n", NULL, @selector(newGame:)); newitem(menu, "New Game", "n", NULL, @selector(newGame:));
@ -1652,7 +1658,7 @@ int main(int argc, char **argv)
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
newitem(menu, "Close", "w", NULL, @selector(performClose:)); newitem(menu, "Close", "w", NULL, @selector(performClose:));
menu = newsubmenu([NSApp mainMenu], "Edit"); menu = newsubmenu([app mainMenu], "Edit");
newitem(menu, "Undo", "z", NULL, @selector(undoMove:)); newitem(menu, "Undo", "z", NULL, @selector(undoMove:));
newitem(menu, "Redo", "S-z", NULL, @selector(redoMove:)); newitem(menu, "Redo", "S-z", NULL, @selector(redoMove:));
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
@ -1662,18 +1668,18 @@ int main(int argc, char **argv)
[menu addItem:[NSMenuItem separatorItem]]; [menu addItem:[NSMenuItem separatorItem]];
newitem(menu, "Solve", "S-s", NULL, @selector(solveGame:)); newitem(menu, "Solve", "S-s", NULL, @selector(solveGame:));
menu = newsubmenu([NSApp mainMenu], "Type"); menu = newsubmenu([app mainMenu], "Type");
typemenu = menu; typemenu = menu;
newitem(menu, "Custom", "", NULL, @selector(customGameType:)); newitem(menu, "Custom", "", NULL, @selector(customGameType:));
menu = newsubmenu([NSApp mainMenu], "Window"); menu = newsubmenu([app mainMenu], "Window");
[NSApp setWindowsMenu: menu]; [app setWindowsMenu: menu];
newitem(menu, "Minimise Window", "m", NULL, @selector(performMiniaturize:)); newitem(menu, "Minimise Window", "m", NULL, @selector(performMiniaturize:));
menu = newsubmenu([NSApp mainMenu], "Help"); menu = newsubmenu([app mainMenu], "Help");
newitem(menu, "Puzzles Help", "?", NSApp, @selector(showHelp:)); newitem(menu, "Puzzles Help", "?", app, @selector(showHelp:));
[NSApp run]; [app run];
[pool release]; [pool release];
return 0; return 0;