From f1222f76521fa7645badaa0c972ad4b2b1dc0b78 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Sebastian=20Markb=C3=A5ge?= Date: Tue, 12 Aug 2025 21:42:24 -0400 Subject: [PATCH] [Fiber] Don't bind retry listener if it's in the cache (#34183) This did an unnecessary bind allocation even if there's cache hit. --- packages/react-reconciler/src/ReactFiberCommitWork.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/packages/react-reconciler/src/ReactFiberCommitWork.js b/packages/react-reconciler/src/ReactFiberCommitWork.js index ba012a67d0..b6716506a0 100644 --- a/packages/react-reconciler/src/ReactFiberCommitWork.js +++ b/packages/react-reconciler/src/ReactFiberCommitWork.js @@ -1894,7 +1894,6 @@ function attachSuspenseRetryListeners( const retryCache = getRetryCache(finishedWork); wakeables.forEach(wakeable => { // Memoize using the boundary fiber to prevent redundant listeners. - const retry = resolveRetryWakeable.bind(null, finishedWork, wakeable); if (!retryCache.has(wakeable)) { retryCache.add(wakeable); @@ -1911,6 +1910,7 @@ function attachSuspenseRetryListeners( } } + const retry = resolveRetryWakeable.bind(null, finishedWork, wakeable); wakeable.then(retry, retry); } });