Merge pull request #2543 from jches/gh/2542

Fix is_hoistable cycle
sveltejs · Apr 25, 2019 · f888123 · f888123
2 parents ed7c32d + 73aca45
commit f888123
Show file tree

Hide file tree

Showing 3 changed files with 17 additions and 1 deletion.
diff --git a/src/compile/Component.ts b/src/compile/Component.ts
@@ -1002,9 +1002,11 @@ export default class Component {
 
 								if (walking.has(other_declaration)) {
 									hoistable = false;
+								} else if (other_declaration.type === 'ExportNamedDeclaration' && walking.has(other_declaration.declaration)) {
+									hoistable = false;
 								} else if (!is_hoistable(other_declaration)) {
 									hoistable = false;
-								}
+                }
 							}
 
 							else {

diff --git a/test/runtime/samples/export-function-hoisting/_config.js b/test/runtime/samples/export-function-hoisting/_config.js
@@ -0,0 +1,3 @@
+export default {
+  html: 'Compile plz'
+}
diff --git a/test/runtime/samples/export-function-hoisting/main.svelte b/test/runtime/samples/export-function-hoisting/main.svelte
@@ -0,0 +1,11 @@
+<script>
+	export function one () {
+    two()
+	}
+
+	export function two () {
+		return one()
+	}
+</script>
+
+Compile plz