Re: [whatwg/webidl] Fix `BufferSource` algorithms for shared and resizable buffers (PR #1529)

@MattiasBuelens commented on this pull request.

Quick self-review.

> @@ -9473,11 +9530,12 @@ a reference to the same object that the IDL value represents.
             <a href="#note-ArrayBuffer-detach-exceptions">explained in that algorithm's
             definition</a>;
         *   |arrayBuffer| is already [=BufferSource/detached=];
-        *   Sufficient memory cannot be allocated in |realm|. Generally this will only be the case
-            if |realm| is in a different [=agent cluster=] than the one in which |arrayBuffer| was
-            allocated. If they are in the same [=agent cluster=], then implementations will just
-            change the backing pointers to get the same observable results with better performance
-            and no allocations.
+        *   Sufficient memory cannot be allocated in the [=current realm=], which is the [=realm=]

I'm not sure if we have to fully spell out the meaning of "current realm" here? Happy to remove that.

> +    which also means that it is not [=BufferSource/detached=]. Otherwise, |view|'s
+    [=BufferSource/byte length=] and [=ArrayBufferView/byte offset=] could throw an exception,
+    and assertions must not be observable.

Okay, this is just the AI copying the review remarks into the specification... 🙄 I'll remove this sentence.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/whatwg/webidl/pull/1529#pullrequestreview-5054351127
You are receiving this because you are subscribed to this thread.

Message ID: <whatwg/webidl/pull/1529/review/5054351127@github.com>