Skip to content

Commit

Permalink
stream: call done() in consistent fashion
Browse files Browse the repository at this point in the history
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>
  • Loading branch information
Trott authored and nodejs-github-bot committed Jul 22, 2021
1 parent f6b2893 commit 44ee6c2
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion lib/internal/webstreams/adapters.js
Expand Up @@ -491,7 +491,7 @@ function newStreamReadableFromReadableStream(readableStream, options = {}) {
done);
return;
}
done(error);
done();
},
});

Expand Down

0 comments on commit 44ee6c2

Please sign in to comment.