Skip to content

Commit 8dd88cc

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 b7748e4 commit 8dd88cc

File tree

1 file changed

+6
-7
lines changed

1 file changed

+6
-7
lines changed

doc/api/stream.md

+6-7
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
@@ -703,7 +703,7 @@ stop until the [`'drain'`][] event is emitted.
703703
While a stream is not draining, calls to `write()` will buffer `chunk`, and
704704
return false. Once all currently buffered chunks are drained (accepted for
705705
delivery by the operating system), the `'drain'` event will be emitted.
706-
It is recommended that once `write()` returns false, no more chunks be written
706+
Once `write()` returns false, do not write more chunks
707707
until the `'drain'` event is emitted. While calling `write()` on a stream that
708708
is not draining is allowed, Node.js will buffer all written chunks until
709709
maximum memory usage occurs, at which point it will abort unconditionally.
@@ -863,10 +863,9 @@ to consume data from a single stream. Specifically, using a combination
863863
of `on('data')`, `on('readable')`, `pipe()`, or async iterators could
864864
lead to unintuitive behavior.
865865

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

872871
#### Class: `stream.Readable`

0 commit comments

Comments
 (0)