Remove unneeded items from state

bryanjos · bryanjos · commit 9d365638912c · 2017-09-02T09:43:52.000-05:00
diff --git a/lib/elixir_script/compiler.ex b/lib/elixir_script/compiler.ex
@@ -25,7 +25,7 @@ defmodule ElixirScript.Compiler do
   @spec compile(atom | [atom], []) :: nil
   def compile(entry_modules, opts \\ []) do
     opts = build_compiler_options(opts, entry_modules)
-    {:ok, pid} = ElixirScript.State.start_link(opts)
+    {:ok, pid} = ElixirScript.State.start_link()
 
     entry_modules = List.wrap(entry_modules)
 
@@ -37,7 +37,7 @@ defmodule ElixirScript.Compiler do
     ElixirScript.Translate.execute(modules, pid)
 
     modules = ElixirScript.State.list_modules(pid)
-    result = ElixirScript.Output.execute(modules, pid)
+    result = ElixirScript.Output.execute(modules, pid, opts)
 
     ElixirScript.State.stop(pid)
 
diff --git a/lib/elixir_script/passes/output.ex b/lib/elixir_script/passes/output.ex
@@ -8,17 +8,15 @@ defmodule ElixirScript.Output do
   @doc """
   Takes outputs the JavaScript code in the specified output
   """
-  @spec execute([atom], pid) :: nil
-  def execute(modules, pid) do
+  @spec execute([atom], pid, map) :: nil
+  def execute(modules, pid, opts) do
     modules = modules
     |> Enum.filter(fn {_, info} -> Map.has_key?(info, :js_ast) end)
     |> Enum.map(fn {_module, info} ->
         info.js_ast
       end
     )
 
-    opts = ModuleState.get_compiler_opts(pid)
-
     js_modules = ModuleState.js_modules(pid)
     |> Enum.filter(fn
       {_module, _name, nil} -> false
diff --git a/lib/elixir_script/state.ex b/lib/elixir_script/state.ex
@@ -3,12 +3,10 @@ defmodule ElixirScript.State do
 
   # Holds the state for the ElixirScript compiler
 
-  def start_link(compiler_opts) do
+  def start_link() do
     Agent.start_link(fn ->
       %{
-        compiler_opts: compiler_opts,
         modules: Keyword.new,
-        refs: [],
         js_modules: []
       }
     end)
@@ -18,12 +16,6 @@ defmodule ElixirScript.State do
     Agent.stop(pid)
   end
 
-  def get_compiler_opts(pid) do
-    Agent.get(pid, fn(state) ->
-      state.compiler_opts
-    end)
-  end
-
   def get_module(pid, module) do
     Agent.get(pid, fn(state) ->
       Keyword.get(state.modules, module)
@@ -90,16 +82,6 @@ defmodule ElixirScript.State do
     end)
   end
 
-  def list_foreign_modules(pid) do
-    Agent.get(pid, fn(state) ->
-      state.modules
-      |> Enum.filter(fn
-        (%{attributes: [__foreign_info__: _]}) -> true
-        (_) -> false
-      end)
-    end)
-  end
-
   def list_modules(pid) do
     Agent.get(pid, fn(state) ->
       state.modules
diff --git a/test/passes/translate/form_test.exs b/test/passes/translate/form_test.exs
@@ -8,7 +8,7 @@ defmodule ElixirScript.Translate.Forms.Test do
 
 
   setup_all do
-    {:ok, pid} = ElixirScript.State.start_link(%{})
+    {:ok, pid} = ElixirScript.State.start_link()
 
     state = %{
       pid: pid
diff --git a/test/passes/translate/forms/bitstring_test.exs b/test/passes/translate/forms/bitstring_test.exs
@@ -4,7 +4,7 @@ defmodule ElixirScript.Translate.Forms.Bitstring.Test do
   alias ESTree.Tools.Builder, as: J
 
   setup_all do
-    {:ok, pid} = ElixirScript.State.start_link(%{})
+    {:ok, pid} = ElixirScript.State.start_link()
 
     state = %{
       pid: pid,
@@ -27,4 +27,4 @@ defmodule ElixirScript.Translate.Forms.Bitstring.Test do
     assert js_ast.right.type == "CallExpression"
   end
 
-end
+end
diff --git a/test/passes/translate/forms/js_test.exs b/test/passes/translate/forms/js_test.exs
@@ -4,7 +4,7 @@ defmodule ElixirScript.Translate.Forms.JS.Test do
   alias ESTree.Tools.Builder, as: J
 
   setup_all do
-    {:ok, pid} = ElixirScript.State.start_link(%{})
+    {:ok, pid} = ElixirScript.State.start_link()
 
     state = %{
       pid: pid,
diff --git a/test/passes/translate/forms/map_test.exs b/test/passes/translate/forms/map_test.exs
@@ -6,7 +6,7 @@ defmodule ElixirScript.Translate.Forms.Map.Test do
   import PropertyTest
 
   setup_all do
-    {:ok, pid} = ElixirScript.State.start_link(%{})
+    {:ok, pid} = ElixirScript.State.start_link()
 
     state = %{
       pid: pid
