stream: remove unused function argument

Trott · web-flow · commit b161440cd3c8 · 2022-01-07T16:49:16.000Z
watchClosed() is only ever called with an anonymous function for the action parameter. This anonymous funciton does not take any arguments. Remove unused argument from invocation of action(). PR-URL: #41403 Reviewed-By: Yongsheng Zhang <zyszys98@gmail.com> Reviewed-By: Mohammed Keyvanzadeh <mohammadkeyvanzade94@gmail.com> Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
diff --git a/lib/internal/webstreams/readablestream.js b/lib/internal/webstreams/readablestream.js
@@ -1332,7 +1332,7 @@ function readableStreamPipeTo(
 
   function watchClosed(stream, promise, action) {
     if (stream[kState].state === 'closed')
-      action(stream[kState].storedError);
+      action();
     else
       PromisePrototypeThen(promise, action, () => {});
   }

Original file line number	Diff line number	Diff line change
`@@ -1332,7 +1332,7 @@ function readableStreamPipeTo(`
`1332`	`1332`
`1333`	`1333`	`function watchClosed(stream, promise, action) {`
`1334`	`1334`	`if (stream[kState].state === 'closed')`
`1335`		`- action(stream[kState].storedError);`
	`1335`	`+ action();`
`1336`	`1336`	`else`
`1337`	`1337`	`PromisePrototypeThen(promise, action, () => {});`
`1338`	`1338`	`}`