Closed Bug 1259544 Opened 7 years ago Closed 7 years ago

Remove Atomics.futexWakeOrRequeue

Categories

(Core :: JavaScript Engine, defect)

Product:
Core
Component:
JavaScript Engine
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

()

Status:
RESOLVED FIXED
Milestone:
mozilla48
Tracking Flags:
Tracking Status
firefox48 --- fixed

People

(Reporter: lth, Assigned: lth)

References

(Blocks 1 open bug)

Details

(Keywords: dev-doc-complete)

Attachments

(1 file)

remove futexWakeOrRequeue
8.94 KB, patch
bbouvier
: review+
Details | Diff | Splinter Review 
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: 7 years ago
status-firefox48: --- → fixed
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?).
Keywords: dev-doc-needed → dev-doc-complete
You need to log in before you can comment on or make changes to this bug.