Re: [whatwg/streams] Standard text for TransformStream (#811)

ricea commented on this pull request.



> -    <li> <code>start(controller)</code> is called immediately, and is typically used to enqueue prefix data that appears
-      in the output but doesn't depend on the input. If this process is asynchronous, it can return a promise to signal
-      success or failure.
-    <li> <code>transform(chunk, controller)</code> is called when a new <a>chunk</a> of input data is ready to be
-      transformed. It can return a promise to signal success or failure of the transformation. The results of the
-      transformation can be appended to the output using the {{ReadableStreamDefaultController/enqueue(chunk)}} method.
-      This permits a single input chunk to result in zero or multiple output chunks. The stream implementation
-      guarantees that <code>transform</code> will be called only after previous transformations have succeeded, and
-      never before <code>start</code> has completed or after <code>flush</code> is called. When no
-      <code>transform</code> method is supplied the identity transform is used, which copies chunks unchanged from the
-      input to the output.
-    <li> <code>flush(controller)</code> is called after all input chunks have been transformed. It is typically used to
-      append a suffix to the output data. If this process is asynchronous, it can return a promise
-      to signal success or failure. On success, the {{readable}} side will be closed.
+    <li><p><code>start(controller)</code> is called immediately, and is typically used to enqueue prefix <a>chunks</a>
+      that appear in the <a>readable side</a> but doesn't depend on any writes to the <a>writable side</a>. If this

"... **don't** depend on ..."

"appear" is awkward here. Maybe "will be read from"?

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

Received on Friday, 13 October 2017 04:26:16 UTC