Closed
Bug 517409
Opened 16 years ago
Closed 16 years ago
IOConnectMethodStructureIStructureO not available to 64-bit application
Categories
(Core :: Widget: Cocoa, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: jaas, Assigned: jaas)
References
Details
Attachments
(1 file)
|
2.81 KB,
patch
|
joelr
:
review+
|
Details | Diff | Splinter Review |
IOConnectMethodStructureIStructureO was deprecated in Mac OS X 10.5 and can't be used by 64-bit applications. We need to find a replacement or make the accelerometer service available only to 32-bit applications using __LP64__ ifdefs. This blocks 64-bit Gecko, bug 468509.
Component: Widget → Widget: Cocoa
QA Contact: general → cocoa
Comment 1•16 years ago
|
||
From Apple migration documentation:
Replace outdated IOConnectMethod* calls—The following functions are not supported in a 64-bit environment:
• IOConnectMethodScalarIScalarO
• IOConnectMethodScalarIStructureO
• IOConnectMethodScalarIStructureI
• IOConnectMethodStructureIStructureO
• IOConnectMethodScalarIScalarO
• IOConnectMethodScalarIStructureO
• IOConnectMethodScalarIStructureI
• IOConnectMethodStructureIStructureO
You should instead use the IOConnectCall* functions:
• IOConnectCallMethod
• IOConnectCallAsyncMethod
• IOConnectCallStructMethod
• IOConnectCallAsyncStructMethod
• IOConnectCallScalarMethod
• IOConnectCallAsyncScalarMethod
This uses the new API when 10.5 is our minimum, the old API when 10.4 is the minimum. It is 64-bit safe and adds a bounds check for the iokit all output.
I don't have a laptop with me right now so this needs to be tested when it is reviewed.
Attachment #401514 -
Flags: review?
Attachment #401514 -
Flags: review? → review?(doug.turner)
Attachment #401514 -
Flags: review?(joelr)
Updated•16 years ago
|
Attachment #401514 -
Flags: review?(joelr) → review+
Attachment #401514 -
Flags: review?(doug.turner)
Comment 3•16 years ago
|
||
looks fine to me too.
pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/aae349de03be
Status: NEW → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
You need to log in
before you can comment on or make changes to this bug.
Description
•