You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Reported in version: HG 2.0 Reported for operating system, platform: iOS 3, iPhone/iPod touch
Comments on the original bug report:
On 2011-06-30 19:03:56 +0000, Joseba García Echebarria wrote:
I just found that SDL is crashing in the iPad simulator for iOS 3.2 but not for iOS 4.2 or over when compiling with Xcode 4.0.2.
The Xcode debugger points to line 127 in video/uikit/SDL_uikitopenglview.m as the line that triggers the crash.
On those lines one can find:
/* Use the main screen scale (for retina display support) */
if ([[UIScreen mainScreen] respondsToSelector:@selector(scale)])
self.contentScaleFactor = [UIScreen mainScreen].scale;
I believe the problem to be that the "scale" selector is supported in iOS 3.2 in the iPad, but contentScaleFactor is not.
I'm no expert in Objective-C, but I believe the following code to be more correct (it doesn't crash for me):
/* Use the main screen scale (for retina display support) */
if ([[UIScreen mainScreen] respondsToSelector:@selector(scale)] && [self respondsToSelector:@selector(contentScaleFactor)])
self.contentScaleFactor = [UIScreen mainScreen].scale;
The same check is being performed in line 155 so I imagine it will crash there, too.
On 2011-08-22 17:58:20 +0000, Vittorio Giovara wrote:
Created attachment 681
fix
yeah, -scale was introduced in 3.2 but made available only in 4.0, weird
anyways we just need to check against contentScaleFactor as we can be sure that both are availble starting from 4.0
the attached patch addresses this
On 2011-11-20 09:24:12 +0000, Vittorio Giovara wrote:
Created attachment 729
updated fix for -contentScaleFactor check
This bug report was migrated from our old Bugzilla tracker.
These attachments are available in the static archive:
fix (screen.patch, text/plain, 2011-08-22 17:58:20 +0000, 1199 bytes)Reported in version: HG 2.0
Reported for operating system, platform: iOS 3, iPhone/iPod touch
Comments on the original bug report:
On 2011-06-30 19:03:56 +0000, Joseba García Echebarria wrote:
On 2011-08-22 17:58:20 +0000, Vittorio Giovara wrote:
On 2011-11-20 09:24:12 +0000, Vittorio Giovara wrote:
On 2012-01-06 22:22:10 +0000, Sam Lantinga wrote:
The text was updated successfully, but these errors were encountered: