search

tc39 / proposal-atomics-wait-async

"asynchronous atomic wait" for ECMAScript
https://tc39.github.io/proposal-atomics-wait-async/
Other
90 stars 18 forks source link

Update Atomics.wait to call DoWait? #21

Closed rwaldron closed 4 years ago

rwaldron commented 4 years ago

With the introduction of the new abstract operation DoWait() that accepts a mode of async or sync, I would expect the algorithm steps in Atomics.wait to be replaced with:

  1. Return DoWait(sync, typedArray, index, value, timeout).
syg commented 4 years ago

Yeah, I guess it should. Originally I preferred not refactoring out a common AO since the branching on sync and async made it harder to read. But... then I committed the refactor anyway? So I must have changed my mind? Been a while.