From fbef6535a5bcb07ea660bbf1c07c65136caa7d88 Mon Sep 17 00:00:00 2001 From: bjarni Date: Sat, 9 Aug 2008 17:56:31 +0000 Subject: (svn r14032) -Fix (r13584): [OSX] changed the condition for selecting 8 or 32 bpp blitter by default Now we will pick 32 bpp if no 8 bpp fullscreen resolutions are available on the main display (the one with the dock) --- src/blitter/factory.hpp | 16 +++++++++++----- 1 file changed, 11 insertions(+), 5 deletions(-) (limited to 'src/blitter') diff --git a/src/blitter/factory.hpp b/src/blitter/factory.hpp index b33c9e5a9..304d73832 100644 --- a/src/blitter/factory.hpp +++ b/src/blitter/factory.hpp @@ -10,6 +10,10 @@ #include "../string_func.h" #include +#if defined(WITH_COCOA) +bool QZ_CanDisplay8bpp(); +#endif /* defined(WITH_COCOA) */ + /** * The base factory, keeping track of all blitters. */ @@ -77,13 +81,15 @@ public: { const char *default_blitter = "8bpp-optimized"; -#if defined(__APPLE__) - /* MacOS X 10.5 removed 8bpp fullscreen support. - * Because of this we will pick 32bpp by default */ - if (MacOSVersionIsAtLeast(10, 5, 0)) { +#if defined(WITH_COCOA) + /* Some people reported lack of fullscreen support in 8 bpp mode. + * While we prefer 8 bpp since it's faster, we will still have to test for support. */ + if (!QZ_CanDisplay8bpp()) { + /* The main display can't go to 8 bpp fullscreen mode. + * We will have to switch to 32 bpp by default. */ default_blitter = "32bpp-anim"; } -#endif /* defined(__APPLE__) */ +#endif /* defined(WITH_COCOA) */ if (GetBlitters().size() == 0) return NULL; const char *bname = (StrEmpty(name)) ? default_blitter : name; -- cgit v1.2.3-54-g00ecf