Add atomic.notify as an alias for atomic.wake

RESOLVED FIXED in Firefox 65

Status

()

enhancement
P3
normal
RESOLVED FIXED
5 months ago
4 months ago

People

(Reporter: lth, Assigned: lth)

Tracking

(Blocks 1 bug, {dev-doc-needed})

Trunk
mozilla65
Points:
---

Firefox Tracking Flags

(firefox65 fixed)

Details

Attachments

(1 attachment)

(Assignee)

Description

5 months ago
An extended bikeshedding session at the Wasm CG video call on Nov 13 decided, in the end, that we will rename atomic.wake as atomic.notify to track what JS is doing and because "wake" and "wait" sound too similar for comfort and tend to confuse listeners.  (Also lots of languages use "notify"; "wake" is a futexism.)

For the time being, let's just add an alias and not worry overmuch about updating everything else.
(Assignee)

Comment 1

5 months ago
Attachment #9025002 - Flags: review?(jseward)
Comment on attachment 9025002 [details] [diff] [review]
bug1507106-atomic-notify-for-wasm.patch

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

Sigh (re bikeshedding etc), although I do think notify is a better name.

Yes.  Looks fine.
Attachment #9025002 - Flags: review?(jseward) → review+

Comment 4

5 months ago
bugherder
https://hg.mozilla.org/mozilla-central/rev/26862c8c9db9
Status: ASSIGNED → RESOLVED
Last Resolved: 5 months ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla65
Not sure if this is an internals thing, or if it is exposed to web devs. Will investigate soon.
Keywords: dev-doc-needed
(Assignee)

Comment 6

5 months ago
Well...  those few people who manipulate textual webassembly (essentially compiler writers) will be exposed to this, but it's not a JS API, it's the name of the instruction when you write wasm text.  And the browser has no means of translating wasm text to wasm binary.  So technically it's not a web API.
OK, that makes sense; I still ought to mention it in (and generally update) the guide to WebAssembly text format on MDN, so I'll keep the dev-doc-needed keyword, but I won't include a note in the Fx65 rel notes.
You need to log in before you can comment on or make changes to this bug.