Skip to content

Commit 5a15cb8

Browse files
Trottjuanarbol
authored andcommitted
doc: clarify recommendations in stream.md
PR-URL: #42555 Reviewed-By: Mestery <[email protected]> Reviewed-By: Tierney Cyren <[email protected]> Reviewed-By: Benjamin Gruenbaum <[email protected]> Reviewed-By: Akhil Marsonya <[email protected]>
1 parent 91a1177 commit 5a15cb8

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

doc/api/stream.md

Lines changed: 6 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -546,8 +546,8 @@ The `writable.uncork()` method flushes all data buffered since
546546
[`stream.cork()`][] was called.
547547

548548
When using [`writable.cork()`][] and `writable.uncork()` to manage the buffering
549-
of writes to a stream, it is recommended that calls to `writable.uncork()` be
550-
deferred using `process.nextTick()`. Doing so allows batching of all
549+
of writes to a stream, defer calls to `writable.uncork()` using
550+
`process.nextTick()`. Doing so allows batching of all
551551
`writable.write()` calls that occur within a given Node.js event loop phase.
552552

553553
```js
@@ -713,7 +713,7 @@ stop until the [`'drain'`][] event is emitted.
713713
While a stream is not draining, calls to `write()` will buffer `chunk`, and
714714
return false. Once all currently buffered chunks are drained (accepted for
715715
delivery by the operating system), the `'drain'` event will be emitted.
716-
It is recommended that once `write()` returns false, no more chunks be written
716+
Once `write()` returns false, do not write more chunks
717717
until the `'drain'` event is emitted. While calling `write()` on a stream that
718718
is not draining is allowed, Node.js will buffer all written chunks until
719719
maximum memory usage occurs, at which point it will abort unconditionally.
@@ -873,10 +873,9 @@ to consume data from a single stream. Specifically, using a combination
873873
of `on('data')`, `on('readable')`, `pipe()`, or async iterators could
874874
lead to unintuitive behavior.
875875

876-
Use of the `readable.pipe()` method is recommended for most users as it has been
877-
implemented to provide the easiest way of consuming stream data. Developers that
878-
require more fine-grained control over the transfer and generation of data can
879-
use the [`EventEmitter`][] and `readable.on('readable')`/`readable.read()`
876+
`readable.pipe()` provides the easiest way to consume stream data. Developers
877+
that require more fine-grained control over the transfer and generation of data
878+
can use the [`EventEmitter`][] and `readable.on('readable')`/`readable.read()`
880879
or the `readable.pause()`/`readable.resume()` APIs.
881880

882881
#### Class: `stream.Readable`

0 commit comments

Comments
 (0)