Closed Bug 1259544 Opened 4 years ago Closed 4 years ago

Remove Atomics.futexWakeOrRequeue

Categories

(Core :: JavaScript Engine, defect)

defect
Not set
normal

Tracking

()

RESOLVED FIXED
mozilla48
Tracking Status
firefox48 --- fixed

People

(Reporter: lth, Assigned: lth)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

The spec changed: the function is obsolete.
Needs to be removed from MDN, too.
Keywords: dev-doc-needed
Another spec change.  Nobody uses this function.
Attachment #8736972 - Flags: review?(bbouvier)
Comment on attachment 8736972 [details] [diff] [review]
remove futexWakeOrRequeue

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

Fare thee well!
Attachment #8736972 - Flags: review?(bbouvier) → review+
For adding some reviewer value here, let's add some links to the specification, explaining what are the motivations for removal (premature optimizations, if I skimmed correctly)

https://github.com/tc39/ecmascript_sharedmem/commit/74f81d97c738ee0cbaace017699f8902fb165888
https://github.com/tc39/ecmascript_sharedmem/issues/11
https://hg.mozilla.org/mozilla-central/rev/88688664f228
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla48
Just nuked the method page from the Atomics reference docs https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Atomics

Not going to add any compatibility note as this is an experimental API anyway and we are shipping it only for 2 releases (will this be uplifted?).
You need to log in before you can comment on or make changes to this bug.