• 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
SDLMain.m 10.6 KB