Creating info map inside of __info__ function to prevent stack overflow

bryanjos · bryanjos · commit 7e1bf3bb3c44 · 2017-09-08T20:27:28.000-05:00
diff --git a/lib/elixir_script/passes/translate/module.ex b/lib/elixir_script/passes/translate/module.ex
@@ -61,9 +61,8 @@ defmodule ElixirScript.Translate.Module do
       _ ->
         { compiled_functions, _ } = Enum.map_reduce(combined_defs, state, &Function.compile(&1, &2))
 
-        info_map = make_info_map(module, state)
-        info_function = make_info_function()
-        compiled_functions = [info_map, info_function] ++ compiled_functions
+        info_function = make_info_function(module, state)
+        compiled_functions = [info_function] ++ compiled_functions
 
         js_ast = ElixirScript.ModuleSystems.Namespace.build(
           module,
@@ -208,7 +207,9 @@ defmodule ElixirScript.Translate.Module do
 
   # Builds the __info__ function that Elixir modules
   # have.
-  defp make_info_function do
+  defp make_info_function(module, state) do
+    info_map = make_info_map(module, state)
+
     get_call = Helpers.call(
       J.member_expression(
         J.identifier("__info__map__"),
@@ -233,6 +234,7 @@ defmodule ElixirScript.Translate.Module do
     )
 
     body = J.block_statement([
+      info_map,
       value,
       body,
       J.throw_statement(
diff --git a/test/integration/integration_test.exs b/test/integration/integration_test.exs
@@ -6,4 +6,9 @@ defmodule ElixirScript.Integration.Test do
     val = call_compiled_function Atom, :to_string, [:atom]
     assert val == "atom"
   end
+
+  test "String interpolation with number" do
+    val = call_compiled_function Integration, :test_string_interpolation, []
+    assert val == "5"
+  end
 end
diff --git a/test/support/integration.ex b/test/support/integration.ex
@@ -0,0 +1,7 @@
+defmodule Integration do
+  @moduledoc false
+
+  def test_string_interpolation do
+    "#{5}"
+  end
+end
