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

@annevk 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>

Newline above this ol.

> +<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>),

You need to move _stream_ here.

Probably needs to say "given a TextDecoderStream object _stream_"

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

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