We are currently migrating Bugzilla to GitHub issues.
Any changes made to the bug tracker now will be lost, so please do not post new bugs or make changes to them.
When we're done, all bug URLs will redirect to their equivalent location on the new bug tracker.

Bug 899

Summary: Fullscreen mouse regression in 10.4, 10.5
Product: SDL Reporter: Jeremiah Morris <bugzilla>
Component: videoAssignee: Ryan C. Gordon <icculus>
Status: RESOLVED FIXED QA Contact: Sam Lantinga <slouken>
Severity: normal    
Priority: P2 CC: wolfy
Version: 1.2.14   
Hardware: All   
OS: Mac OS X 10.4 (PPC)   
Attachments: Re-enable mouseLocation workaround on 10.4, 10.5

Description Jeremiah Morris 2009-12-09 16:23:50 UTC
Created attachment 459 [details]
Re-enable mouseLocation workaround on 10.4, 10.5

OS X systems before 10.6 have a bug with [NSEvent mouseLocation] if the screen
resolution changes. SDL_QuartzVideo.m contains a workaround for this bug, but
it was placed inside an #ifdef in revision 4762. The comment reads, "I'm
gambling they fixed this by 10.4."

After seeing this bug on several Tiger and Leopard systems (both PPC and
Intel), I can confirm that it's not fixed until 10.6. The workaround doesn't
compile for x86_64/10.6, so I can understand why it was segregated, but it
needs to remain in place for the i386 and ppc versions. The workaround causes
no problems under 10.6, even though it's not necessary there.

Patch is attached with one method of keeping the workaround active on the
systems that need it.
Comment 1 Sam Lantinga 2009-12-11 07:13:01 UTC
Your fix is in subversion, thanks! :)