We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 97 - setAppleMenu method on newer OS X SDK
Summary: setAppleMenu method on newer OS X SDK
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: main (show other bugs)
Version: HG 1.2
Hardware: PowerPC Mac OS X (All)
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2006-01-27 10:53 UTC by Ryan C. Gordon
Modified: 2006-03-22 17:33 UTC (History)
1 user (show)

See Also:

Attachments
Fix warnings in SDLMain.m (1.23 KB, patch)
2006-03-22 13:10 UTC, Max Horn 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Ryan C. Gordon 2006-01-27 10:53:55 UTC 
Compiler warning on the Intel iMacs...

macosx/SDLMain.m: In function '-[SDLMain setupWorkingDirectory:]':
macosx/SDLMain.m:85: warning: pointer targets in passing argument 3 of 'CFURLGetFileSystemRepresentation' differ in signedness
macosx/SDLMain.m: In function 'setApplicationMenu':
macosx/SDLMain.m:158: warning: no '-setAppleMenu:' method found
macosx/SDLMain.m:158: warning: (Messages without a matching method signature
macosx/SDLMain.m:158: warning: will be assumed to return 'id' and accept
macosx/SDLMain.m:158: warning: '...' as arguments.)


--ryan.
Comment 1 Ryan C. Gordon 2006-01-27 11:23:28 UTC 
Setting Sam as "QA Contact" on all bugs (even resolved ones) so he'll definitely be in the loop to any further discussion here about SDL.

--ryan.
Comment 2 Max Horn 2006-03-22 13:10:38 UTC 
Created attachment 91 [details]
Fix warnings in SDLMain.m

See here for a discussion about the "setAppleMenu" warning: <http://www.allegro.cc/forums/thread/497928>.

The attached patch fixes all warning.
Comment 3 Ryan C. Gordon 2006-03-22 17:33:37 UTC 
Patch is now in CVS, thanks!

--ryan.