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 - Fullscreen mouse regression in 10.4, 10.5
Summary: Fullscreen mouse regression in 10.4, 10.5
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: video (show other bugs)
Version: 1.2.14
Hardware: All Mac OS X 10.4 (PPC)
: P2 normal
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-09 16:23 UTC by Jeremiah Morris
Modified: 2009-12-11 07:13 UTC (History)
1 user (show)

See Also:

Attachments
Re-enable mouseLocation workaround on 10.4, 10.5 (567 bytes, patch)
2009-12-09 16:23 UTC, Jeremiah Morris 		Details | Diff
View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
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! :)