From 370194796e6daaf74af7dcfc3be469d2f2dc0a88 Mon Sep 17 00:00:00 2001
From: Anton Bannykh <Anton.Bannykh@jetbrains.com>
Date: Thu, 7 Feb 2019 20:35:47 +0300
Subject: [PATCH] JS: fix synthethic name bindings for the inlined local
 declarations

TODO add synthetic order-based namebindings for inlined local
declarations AFTER all post-processings. This would be a more robust
approach imho
---
 .../src/org/jetbrains/kotlin/js/inline/util/collectUtils.kt    | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

diff --git a/js/js.inliner/src/org/jetbrains/kotlin/js/inline/util/collectUtils.kt b/js/js.inliner/src/org/jetbrains/kotlin/js/inline/util/collectUtils.kt
index 46524a4095a..0118623b5a3 100644
--- a/js/js.inliner/src/org/jetbrains/kotlin/js/inline/util/collectUtils.kt
+++ b/js/js.inliner/src/org/jetbrains/kotlin/js/inline/util/collectUtils.kt
@@ -136,8 +136,9 @@ fun collectDefinedNamesInAllScopes(scope: JsNode): Set<JsName> {
 
         override fun visitFunction(x: JsFunction) {
             super.visitFunction(x)
-            x.name?.let { names += it }
+            // TODO This is a hack
             names += x.parameters.map { it.name }
+            x.name?.let { names += it }
         }
 
         override fun visitBreak(x: JsBreak) {