Closed
Bug 391467
Opened 18 years ago
Closed 18 years ago
ActionMonkey: Clean up js_NewDouble (remove gcflag, and check Lock)
Categories
(Core :: JavaScript Engine, defect)
Tracking
()
RESOLVED
FIXED
People
(Reporter: Mardak, Assigned: Mardak)
Details
Attachments
(1 file)
|
6.32 KB,
patch
|
jorendorff
:
review+
|
Details | Diff | Splinter Review |
gcflags aren't needed for js_NewDoubles anymore and js_LockGCThing could potentially not lock the double, so check it.
Attachment #275891 -
Flags: review?(jorendorff)
|
||
Comment 1•18 years ago
|
||
Comment on attachment 275891 [details] [diff] [review]
v1
Looks pretty straightforward. Nice "en passant" bugfix.
Attachment #275891 -
Flags: review?(jorendorff) → review+
|
Assignee | |
Comment 2•18 years ago
|
||
pushed to hg/actionmonkey
remote: added 1 changesets with 4 changes to 4 files
changeset 4356: 0259991ad83a
Status: ASSIGNED → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
|
||
Updated•18 years ago
|
Flags: in-testsuite-
You need to log in
before you can comment on or make changes to this bug.
Description
•