@samitouri / QOS-React-2 / commits / 168da8d557

Fix typo that happened during rebasing

I changed the type of this functions returned value but forgot to change the check. It happens to work before anyway, because eventually the interleaved updates will get transferred at the beginning of the next render phase. But this is more logically consistent.

Andrew Clark committed Apr 12, 2022 at 15:41 UTC 168da8d55782f3b34e2a6aa0c4dd0587696afdbd
2 files changed +2 -2
packages/react-reconciler/src/ReactFiberWorkLoop.new.js
+1 -1
@@ -732,7 +732,7 @@ export function isInterleavedUpdate(fiber: Fiber, lane: Lane) {
732 // defensive coding measure in case a new update comes in between when
733 // rendering has finished and when the interleaved updates are transferred
734 // to the main queue.
735 - hasInterleavedUpdates() !== null) &&
735 + hasInterleavedUpdates()) &&
736 (fiber.mode & ConcurrentMode) !== NoMode &&
737 // If this is a render phase update (i.e. UNSAFE_componentWillReceiveProps),
738 // then don't treat this as an interleaved update. This pattern is
packages/react-reconciler/src/ReactFiberWorkLoop.old.js
+1 -1
@@ -732,7 +732,7 @@ export function isInterleavedUpdate(fiber: Fiber, lane: Lane) {
732 // defensive coding measure in case a new update comes in between when
733 // rendering has finished and when the interleaved updates are transferred
734 // to the main queue.
735 - hasInterleavedUpdates() !== null) &&
735 + hasInterleavedUpdates()) &&
736 (fiber.mode & ConcurrentMode) !== NoMode &&
737 // If this is a render phase update (i.e. UNSAFE_componentWillReceiveProps),
738 // then don't treat this as an interleaved update. This pattern is