Closed Bug 1140336 Opened 10 years ago Closed 10 years ago

Dubious assert in VMFunction.init when called from the copy ctor

Categories

(Core :: JavaScript Engine: JIT, defect)

x86_64
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla39
Tracking Status
firefox39 --- fixed

People

(Reporter: bbouvier, Assigned: bbouvier)

References

(Blocks 1 open bug)

Details

(Keywords: coverity, Whiteboard: [CID 1286215])

Attachments

(1 file)

Attached patch dubious.assert.patch (deleted) — Splinter Review
DXR says VMFunction::init is used only in the copy ctor of VMFunction, so let's just merge these two, if that's possible, and remove the assert.

https://treeherder.mozilla.org/#/jobs?repo=try&revision=a32ab237b4f8
It builds on all platforms! Got r+ from hannes on irc:
https://hg.mozilla.org/integration/mozilla-inbound/rev/1c397ab57466
https://hg.mozilla.org/mozilla-central/rev/1c397ab57466
Status: NEW → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla39
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: