Closed Bug 887906 Opened 7 years ago Closed 7 years ago

use the correct ctypes types for long and short function parameters in JNI.jsm

Categories

(Firefox for Android :: General, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
Firefox 25

People

(Reporter: froydnj, Unassigned)

Details

Attachments

(1 file)

long might be 32-bit or 64-bit, depending on the architecture.  short is usually
16 bits, but we might as well specify a 16-bit type directly.
Comment on attachment 768441 [details] [diff] [review]
use the correct ctypes types for long and short function parameters in JNI.jsm

Review of attachment 768441 [details] [diff] [review]:
-----------------------------------------------------------------

nice.
Attachment #768441 - Flags: review?(wjohnston) → review+
https://hg.mozilla.org/mozilla-central/rev/5debf3545396
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → FIXED
Target Milestone: --- → Firefox 25
You need to log in before you can comment on or make changes to this bug.