• Ryan C. Gordon's avatar
    To: sdl@libsdl.org · a499115a
    Ryan C. Gordon authored
    From: Christian Walther <cwalther@gmx.ch>
    Date: Wed, 28 Dec 2005 12:13:20 +0100
    Subject: [SDL] Fix for opening documents on Mac OS X < 10.4
    
    The current code in SDLMain.m that transforms documents opened from the
    Finder into command-line arguments (introduced in revision 1.14,
    2005-08-11) uses the methods -[NSString lengthOfBytesUsingEncoding:] and
    -[NSString getCString:maxLength:encoding:], which are only available in
    Mac OS X 10.4.
    
    Compiling this code on 10.3 produces warnings, and running it (i.e.
    starting an SDL application by opening a document) leads to weird
    behavior which I didn't investigate in detail ("*** -[NSCFString
    lengthOfBytesUsingEncoding:]: selector not recognized" is printed to the
    console log, and the SDL window never opens).
    
    The attached patch removes the offending calls and uses -[NSString
    UTF8String] instead, which is available everywhere. Tested on 10.3.9,
    and I see no reason why it shouldn't also work on 10.2 and 10.4.
    
    Two further comments:
    
    * The comment above the -[SDLMain application: openFile:] implementation
    says "You need to have a CFBundleDocumentsType section in your
    Info.plist to get this message, apparently." This is not the case in my
    experience - it worked just fine with a hand-built bare-bones
    application consisting only of Test.app/Contents/MacOS/test, without any
    Info.plist (although you have to press the option and command keys for
    such an application to accept a dragged file).
    
    * I took the liberty of cleaning up another area of SDLMain.m: I changed
    "CustomApplicationMain (argc, argv)" to "CustomApplicationMain (int
    argc, char **argv)". This avoids the "type of `argv' defaults to `int'"
    warnings, and I'm not sure if leaving out the types could cause problems
    on platforms where an int and a char** aren't of the same size.
    
      -Christian
    
    --HG--
    extra : convert_revision : svn%3Ac70aab31-4412-0410-b14c-859654838e24/trunk%401214
    a499115a
Name
Last commit
Last update
..
audio Loading commit data...
cdrom Loading commit data...
cpuinfo Loading commit data...
endian Loading commit data...
events Loading commit data...
file Loading commit data...
hermes Loading commit data...
joystick Loading commit data...
loadso Loading commit data...
main Loading commit data...
thread Loading commit data...
timer Loading commit data...
video Loading commit data...
.cvsignore Loading commit data...
Makefile.am Loading commit data...
Makefile.wat Loading commit data...
SDL.c Loading commit data...
SDL_error.c Loading commit data...
SDL_error_c.h Loading commit data...
SDL_fatal.c Loading commit data...
SDL_fatal.h Loading commit data...
SDL_getenv.c Loading commit data...
SDL_loadso.c Loading commit data...