Remove unused returnFiber (#28504)

bubucuo · web-flow · commit 64f354cf27a1 · 2024-03-06T14:02:51.000-05:00
There is no use of `returnFiber` here, it can be removed.
diff --git a/packages/react-reconciler/src/ReactChildFiber.js b/packages/react-reconciler/src/ReactChildFiber.js
@@ -438,7 +438,6 @@ function createChildReconciler(
   }
 
   function mapRemainingChildren(
-    returnFiber: Fiber,
     currentFirstChild: Fiber,
   ): Map<string | number, Fiber> {
     // Add the remaining children to a temporary map so that we can find them by
@@ -1194,7 +1193,7 @@ function createChildReconciler(
     }
 
     // Add all children to a key map for quick lookups.
-    const existingChildren = mapRemainingChildren(returnFiber, oldFiber);
+    const existingChildren = mapRemainingChildren(oldFiber);
 
     // Keep scanning and use the map to restore deleted items as moves.
     for (; newIdx < newChildren.length; newIdx++) {
@@ -1404,7 +1403,7 @@ function createChildReconciler(
     }
 
     // Add all children to a key map for quick lookups.
-    const existingChildren = mapRemainingChildren(returnFiber, oldFiber);
+    const existingChildren = mapRemainingChildren(oldFiber);
 
     // Keep scanning and use the map to restore deleted items as moves.
     for (; !step.done; newIdx++, step = newChildren.next()) {

Original file line number	Diff line number	Diff line change
`@@ -438,7 +438,6 @@ function createChildReconciler(`
`438`	`438`	`}`
`439`	`439`
`440`	`440`	`function mapRemainingChildren(`
`441`		`- returnFiber: Fiber,`
`442`	`441`	`currentFirstChild: Fiber,`
`443`	`442`	`): Map<string \| number, Fiber> {`
`444`	`443`	`// Add the remaining children to a temporary map so that we can find them by`
`@@ -1194,7 +1193,7 @@ function createChildReconciler(`
`1194`	`1193`	`}`
`1195`	`1194`
`1196`	`1195`	`// Add all children to a key map for quick lookups.`
`1197`		`- const existingChildren = mapRemainingChildren(returnFiber, oldFiber);`
	`1196`	`+ const existingChildren = mapRemainingChildren(oldFiber);`
`1198`	`1197`
`1199`	`1198`	`// Keep scanning and use the map to restore deleted items as moves.`
`1200`	`1199`	`for (; newIdx < newChildren.length; newIdx++) {`
`@@ -1404,7 +1403,7 @@ function createChildReconciler(`
`1404`	`1403`	`}`
`1405`	`1404`
`1406`	`1405`	`// Add all children to a key map for quick lookups.`
`1407`		`- const existingChildren = mapRemainingChildren(returnFiber, oldFiber);`
	`1406`	`+ const existingChildren = mapRemainingChildren(oldFiber);`
`1408`	`1407`
`1409`	`1408`	`// Keep scanning and use the map to restore deleted items as moves.`
`1410`	`1409`	`for (; !step.done; newIdx++, step = newChildren.next()) {`