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 861 - 64-bit expects mForkRefNum to be int
Summary: 64-bit expects mForkRefNum to be int
Status: RESOLVED FIXED
Alias: None
Product: SDL
Classification: Unclassified
Component: cdrom (show other bugs)
Version: 1.2.14
Hardware: x86_64 Mac OS X 10.6
: P2 major
Assignee: Ryan C. Gordon
QA Contact: Sam Lantinga
URL:
Keywords: target-1.2.14
Depends on:
Blocks:
 
Reported: 2009-10-19 19:16 UTC by C.W. Betts
Modified: 2009-10-19 20:47 UTC (History)
1 user (show)

See Also:

Attachments
patch to make CD work on OS X (462 bytes, patch)
2009-10-19 19:16 UTC, C.W. Betts 		Details | Diff
Fix CD player on 64-bit Mac (1.06 KB, patch)
2009-10-19 19:56 UTC, C.W. Betts 		Details | Diff
Show Obsolete (1) 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 C.W. Betts 2009-10-19 19:16:41 UTC 
Created attachment 429 [details]
patch to make CD work on OS X

In 64-bit libraries, the fork reference is a different size than in 32-bit. 
Apple has tried to help this transition by making a typedef called FSIORefNum,
but it is only available in Mac OS X 10.5 headers or later.  I first noticed
this bug in Mac OS X's CD player code, and the patch is set to fix it
accordingly.  This might also be an issue in SDL_QuartzVideo.m
Comment 1 C.W. Betts 2009-10-19 19:56:28 UTC 
Created attachment 430 [details]
Fix CD player on 64-bit Mac

heh, whoops!  This patch actually does fix the issue and compiles.
Comment 2 Sam Lantinga 2009-10-19 20:47:41 UTC 
Thanks!  Your fix is in for release.