Move back to function

Author: JoviDeCroock

src/diff/index.js

@@ -13,7 +13,7 @@ import { BaseComponent, getDomSibling } from '../component';
 import { Fragment } from '../create-element';
 import { diffChildren } from './children';
 import { setProperty } from './props';
-import { assign, isArray, slice } from '../util';
+import { assign, isArray, removeNode, slice } from '../util';
 import options from '../options';
 
 /**
@@ -346,8 +346,7 @@ export function diff(
 					newVNode._dom = oldDom;
 				} else {
 					for (let i = excessDomChildren.length; i--; ) {
-						const child = excessDomChildren[i];
-						if (child) child.remove();
+						removeNode(excessDomChildren[i]);
 					}
 				}
 			} else {
@@ -604,8 +603,7 @@ function diffElementNodes(
 			// Remove children that are not part of any vnode.
 			if (excessDomChildren != NULL) {
 				for (i = excessDomChildren.length; i--; ) {
-					const child = excessDomChildren[i];
-					if (child) child.remove();
+					removeNode(excessDomChildren[i]);
 				}
 			}
 		}
@@ -704,8 +702,8 @@ export function unmount(vnode, parentVNode, skipRemove) {
 		}
 	}
 
-	if (!skipRemove && vnode._dom != null && vnode._dom.parentNode) {
-		vnode._dom.remove();
+	if (!skipRemove) {
+		removeNode(vnode._dom);
 	}
 
 	vnode._component = vnode._parent = vnode._dom = UNDEFINED;

src/util.js

@@ -15,3 +15,11 @@ export function assign(obj, props) {
 	for (let i in props) obj[i] = props[i];
 	return /** @type {O & P} */ (obj);
 }
+
+/**
+ * Remove a child node from its parent if attached.
+ * @param {import('./internal').PreactElement | null} node The node to remove
+ */
+export function removeNode(node) {
+	if (node && node.parentNode) node.remove();
+}