stream: simplify setPromiseHandled utility

aduh95 · web-flow · commit 5e61a07d0356 · 2026-05-01T22:08:25.000Z
Signed-off-by: Antoine du Hamel <duhamelantoine1995@gmail.com> PR-URL: #63032 Reviewed-By: Michaël Zasso <targos@protonmail.com> Reviewed-By: Matteo Collina <matteo.collina@gmail.com>
diff --git a/lib/internal/webstreams/util.js b/lib/internal/webstreams/util.js
@@ -178,7 +178,7 @@ function setPromiseHandled(promise) {
   // MarkAsHandled, but this avoids the extra boundary cross
   // and is hopefully faster at the cost of an extra Promise
   // allocation.
-  PromisePrototypeThen(promise, () => {}, () => {});
+  PromisePrototypeThen(promise, undefined, () => {});
 }
 
 async function nonOpFlush() {}

Original file line number	Diff line number	Diff line change
`@@ -178,7 +178,7 @@ function setPromiseHandled(promise) {`
`178`	`178`	`// MarkAsHandled, but this avoids the extra boundary cross`
`179`	`179`	`// and is hopefully faster at the cost of an extra Promise`
`180`	`180`	`// allocation.`
`181`		`- PromisePrototypeThen(promise, () => {}, () => {});`
	`181`	`+ PromisePrototypeThen(promise, undefined, () => {});`
`182`	`182`	`}`
`183`	`183`
`184`	`184`	`async function nonOpFlush() {}`