Re: [whatwg/encoding] Editorial: separate `TextDecoderStream` constructor to two algos (PR #364)

@MattiasBuelens commented on this pull request.



>  
- <li><p>Set <a>this</a>'s <a for=TextDecoderCommon>ignore BOM</a> to
- <var>options</var>["{{TextDecoderOptions/ignoreBOM}}"].
+<div algorithm>
+<p>To <dfn export>set up a text decoder stream</dfn> <var>stream</var>, given a
+{{TextDecoderStream}} stream, an optional [=/encoding=] <var>encoding</var> (default <a>UTF-8</a>),
+an optional <a for=/>error mode</a> <var>errorMode</var> (default "<code>replacement</code>"), and
+an optional boolean <var>ignoreBOM</var> (default false), run these steps:
+<ol>
+ <li><p><a>Assert</a>: <var>encoding</var> is not <code>replacement</code>.

In this case, we want to refer to the [replacement encoding](https://encoding.spec.whatwg.org/#replacement), not to the [`"replacement"` error mode](https://encoding.spec.whatwg.org/#error-mode).

```suggestion
 <li><p><a>Assert</a>: <var>encoding</var> is not <a>replacement</a>.
```

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

Message ID: <whatwg/encoding/pull/364/review/4328773319@github.com>