If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Remove MaybeConstructor using CallTyped

RESOLVED FIXED in Firefox 41

Status

()

Core
JavaScript: GC
RESOLVED FIXED
2 years ago
2 years ago

People

(Reporter: terrence, Assigned: terrence)

Tracking

(Blocks: 1 bug)

Trunk
mozilla41
Points:
---

Firefox Tracking Flags

(firefox41 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

2 years ago
Created attachment 8608903 [details] [diff] [review]
use_calltyped_to_simplify_MaybeConstructor-v0.diff

As promised, we can now rip out MaybeConstructor and just call T::maybeConstructor "directly".
Attachment #8608903 - Flags: review?(jcoppeard)

Updated

2 years ago
Attachment #8608903 - Flags: review?(jcoppeard) → review+
(Assignee)

Comment 1

2 years ago
Wow, I really just typed Constructor everywhere Compartment should have gone. Must have had my mind on the /next/ patch in the series when filing this bug. :-P
(Assignee)

Comment 2

2 years ago
https://treeherder.mozilla.org/#/jobs?repo=try&revision=302777514294

Comment 3

2 years ago
https://hg.mozilla.org/integration/mozilla-inbound/rev/390550f161da
https://hg.mozilla.org/mozilla-central/rev/390550f161da
Status: ASSIGNED → RESOLVED
Last Resolved: 2 years ago
status-firefox41: affected → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla41
You need to log in before you can comment on or make changes to this bug.