Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

Commit 44ee6c2

Browse filesBrowse files
Trottnodejs-github-bot
authored andcommitted
stream: call done() in consistent fashion
All three `destroy()` functions defined in adapter.js contain a `done()` function that takes no arguments. In one instance, however, an argument is nonetheless passed. Remove it for consistency. PR-URL: #39475 Reviewed-By: James M Snell <jasnell@gmail.com> Reviewed-By: Tobias Nießen <tniessen@tnie.de> Reviewed-By: Evan Lucas <evanlucas@me.com>
1 parent f6b2893 commit 44ee6c2
Copy full SHA for 44ee6c2

File tree

Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Filter options
Expand file treeCollapse file tree

1 file changed

+1
-1
lines changed
Open diff view settings
Collapse file

‎lib/internal/webstreams/adapters.js‎

Copy file name to clipboardExpand all lines: lib/internal/webstreams/adapters.js
+1-1Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -491,7 +491,7 @@ function newStreamReadableFromReadableStream(readableStream, options = {}) {
491491
done);
492492
return;
493493
}
494-
done(error);
494+
done();
495495
},
496496
});
497497

0 commit comments

Comments
0 (0)
Morty Proxy This is a proxified and sanitized view of the page, visit original site.