Fix jsctypes unsigned __int64 conversion on windows

RESOLVED FIXED in mozilla1.9.3a1

Status

()

defect
RESOLVED FIXED
10 years ago
10 years ago

People

(Reporter: dwitte, Assigned: dwitte)

Tracking

Trunk
mozilla1.9.3a1
x86
Windows XP
Points:
---

Firefox Tracking Flags

(status1.9.2 beta1-fixed)

Details

Attachments

(1 attachment)

See bug 513783 comment 54 and bug 513783 comment 55. We can specialize the templated jsvalToIntStrict() function for unsigned __int64 on windows.
Hardware: x86_64 → x86
Posted patch v1Splinter Review
Passes tests on tryserver. (The source I quoted in bug 513783 comment 55 claimed that MSVC also throws a compile error and/or has runtime issues converting unsigned __int64 to double, but I'm happily surprised that it doesn't, otherwise the same unit test should have failed.)
Assignee: nobody → dwitte
Attachment #403410 - Flags: review?(jorendorff)
Comment on attachment 403410 [details] [diff] [review]
v1

Amazing.
Attachment #403410 - Flags: review?(jorendorff) → review+
Attachment #403410 - Flags: approval1.9.2?
Product: Other Applications → Core
Version: Trunk → unspecified
http://hg.mozilla.org/mozilla-central/rev/1d1373e850f0
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Attachment #403410 - Flags: approval1.9.2? → approval1.9.2+
Target Milestone: --- → mozilla1.9.3a1
Version: unspecified → Trunk
You need to log in before you can comment on or make changes to this bug.