New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Improve OSX bit rot #6695
Improve OSX bit rot #6695
Conversation
Compiles and runs for me on macOS 10.12.6. I'm not competent to review the changes. I could try and trigger specific issues that the commits address if that helps. |
src/fontdetection.cpp
Outdated
|
||
/* Get a file system reference for the font. */ | ||
FSRef ref; | ||
/* Get font reference from name. */ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation
src/fontdetection.cpp
Outdated
CFStringRef name = CFStringCreateWithCString(kCFAllocatorDefault, font_name, kCFStringEncodingUTF8); | ||
|
||
#if (MAC_OS_X_VERSION_MAX_ALLOWED >= MAC_OS_X_VERSION_10_6) | ||
if (MacOSVersionIsAtLeast(10, 6, 0)) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indent
src/fontdetection.cpp
Outdated
CFRelease(fontURL); | ||
} | ||
if (descs != NULL) CFRelease(descs); | ||
} else |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indent
697ef11
to
1372a00
Compare
Now with less spaces :) |
1372a00
to
d184b72
Compare
Exterminate all the spaces! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Like any of us has a clue; sounds good to me! :D
Tnx for taking care of Mac OS X :)
OSX APIs are an ever-moving target. Update some uses in OTTD to hopefully improve compatibility with newer OSX versions.