Remove TODO comments

preactjs · Feb 17, 2020 · 19eaa94 · 19eaa94
1 parent 3ede3a7
commit 19eaa94
Showing 1 changed file with 2 additions and 3 deletions.
diff --git a/src/diff/children.js b/src/diff/children.js
@@ -140,7 +140,6 @@ export function diffChildren(
 						// can clean up the property
 						childVNode._nextDom = undefined;
 
-						// TODO: Investigate removing this `if` to save bytes
 						if (oldVNode._newIndex != null) {
 							oldChildren[oldVNode._newIndex] = undefined;
 						}
@@ -172,8 +171,8 @@ export function diffChildren(
 
 							// Since this code path means oldVNode moved up in the tree,
 							// remove oldVNode from oldChildren so that getDomSibling doesn't see it
-							// when searching for siblings
-							// TODO: Investigate removing this `if` to save bytes
+							// when searching for siblings. This mimics actually moving the DOM like
+							// insertBefore would do
 							if (oldVNode._newIndex != null) {
 								oldChildren[oldVNode._newIndex] = undefined;
 							}