simplify

Author: Rich-Harris

packages/svelte/src/compiler/phases/3-transform/client/visitors/template.js

@@ -1072,7 +1072,7 @@ function create_block(parent, name, nodes, context) {
 
 			body.push(...state.before_init, ...state.init);
 
-			close = b.stmt(b.call('$.close_frag', b.id('$$anchor'), id));
+			close = b.stmt(b.call('$.close', b.id('$$anchor'), id));
 		}
 	} else {
 		body.push(...state.before_init, ...state.init);

packages/svelte/src/internal/client/dom/template.js

@@ -155,16 +155,15 @@ export const comment = template('<!>', TEMPLATE_FRAGMENT);
 /**
  * Assign the created (or in hydration mode, traversed) dom elements to the current block
  * and insert the elements into the dom (in client mode).
- * @param {import('#client').Dom} dom
- * @param {boolean} is_fragment
  * @param {Text | Comment | Element} anchor
+ * @param {import('#client').Dom} dom
  * @returns {import('#client').Dom}
  */
-function close_template(dom, is_fragment, anchor) {
+export function close(anchor, dom) {
 	var current = dom;
 
 	if (!hydrating) {
-		if (is_fragment) {
+		if (/** @type {Node} */ (dom).nodeType === 11) {
 			// if hydrating, `dom` is already an array of nodes, but if not then
 			// we need to create an array to store it on the current effect
 			current = /** @type {import('#client').Dom} */ ([.../** @type {Node} */ (dom).childNodes]);
@@ -178,19 +177,3 @@ function close_template(dom, is_fragment, anchor) {
 
 	return current;
 }
-
-/**
- * @param {Text | Comment | Element} anchor
- * @param {Element | Text} dom
- */
-export function close(anchor, dom) {
-	return close_template(dom, false, anchor);
-}
-
-/**
- * @param {Text | Comment | Element} anchor
- * @param {Element | Text} dom
- */
-export function close_frag(anchor, dom) {
-	return close_template(dom, true, anchor);
-}

packages/svelte/tests/snapshot/samples/bind-this/_expected/client/index.svelte.js

@@ -11,6 +11,6 @@ export default function Bind_this($$anchor, $$props) {
 	var node = $.first_child(fragment);
 
 	$.bind_this(Foo(node, {}), ($$value) => foo = $$value, () => foo);
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }

packages/svelte/tests/snapshot/samples/dynamic-attributes-casing/_expected/client/main.svelte.js

@@ -33,6 +33,6 @@ export default function Main($$anchor, $$props) {
 		$.set_custom_element_data(custom_element, "fooBar", x);
 	});
 
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }

packages/svelte/tests/snapshot/samples/each-string-template/_expected/client/index.svelte.js

@@ -17,6 +17,6 @@ export default function Each_string_template($$anchor, $$props) {
 		return $.close($$anchor, text);
 	});
 
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }

packages/svelte/tests/snapshot/samples/function-prop-no-getter/_expected/client/index.svelte.js

@@ -28,6 +28,6 @@ export default function Function_prop_no_getter($$anchor, $$props) {
 		}
 	});
 
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }

packages/svelte/tests/snapshot/samples/state-proxy-literal/_expected/client/index.svelte.js

@@ -31,7 +31,7 @@ export default function State_proxy_literal($$anchor, $$props) {
 	button.__click = [reset, str, tpl];
 	$.bind_value(input, () => $.get(str), ($$value) => $.set(str, $$value));
 	$.bind_value(input_1, () => $.get(tpl), ($$value) => $.set(tpl, $$value));
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }
 

packages/svelte/tests/snapshot/samples/svelte-element/_expected/client/index.svelte.js

@@ -11,6 +11,6 @@ export default function Svelte_element($$anchor, $$props) {
 	var node = $.first_child(fragment);
 
 	$.element(node, tag, false);
-	$.close_frag($$anchor, fragment);
+	$.close($$anchor, fragment);
 	$.pop();
 }