Re: [whatwg/webidl] Editorial: Define the WebIDL string types in terms of Infra types. (PR #1287)

@annevk approved this pull request.

Thanks again for tackling this, really appreciate it!

>  Such sequences might be interpreted as UTF-8 encoded strings [[!RFC3629]]
 or strings in some other 8-bit-per-code-unit encoding, although this is not required.

I wonder if we should remove this. In particular because when something returns a ByteString it's not expected that would not be treated as an isomorphic string. (Isomorphic string is indeed not defined at the moment, but perhaps at some point we should add it to Infra as the equivalent to the 8-bit strings implementations have when ASCII string isn't sufficient.)

Okay to leave this for a follow-up though.

> @@ -5845,10 +5843,10 @@ can be set to [=value of string literal tokens|the value=] of a
 <h4 oldids="dom-USVString" id="idl-USVString" interface>USVString</h4>
 
 The {{USVString}} type
-corresponds to the set of all possible sequences of
-[=scalar values=],
-which are all of the Unicode code points apart from the
-surrogate code points.
+corresponds to [=scalar value strings=].
+Depending on the context,
+these can be treated as sequences of either 16-bit unsigned integer [=code units=]
+or [=scalar value=] [=code points=].

```suggestion
or [=scalar values=].
```
(Otherwise we're doing the near equivalent of a round circle.)

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

Message ID: <whatwg/webidl/pull/1287/review/1395179475@github.com>

Received on Friday, 21 April 2023 07:10:11 UTC