Get rid of the r helper since it causes conflicts related to modules dependencies

Author: José Valim

lib/iex/lib/iex/helpers.ex

@@ -22,7 +22,6 @@ defmodule IEx.Helpers do
   * `ls/1`    — lists the contents of the specified directory
   * `m/0`     — prints loaded modules
   * `pwd/0`   — prints the current working directory
-  * `r/0`     — recompile and reload all modules that were previously reloaded
   * `r/1`     — recompiles and reloads the given module's source file
   * `s/1`     — prints spec information
   * `t/1`     — prints type information
@@ -278,14 +277,6 @@ defmodule IEx.Helpers do
     IEx.History.nth(n).result
   end
 
-  @doc """
-  Reloads all modules that have already been reloaded with `r/1` at any point
-  in the current IEx session.
-  """
-  def r do
-    List.flatten(Enum.map(iex_reloaded, &do_r(&1)))
-  end
-
   @doc """
   Recompiles and reloads the specified module's source file.
 
@@ -306,11 +297,9 @@ defmodule IEx.Helpers do
         :nosource
 
       String.ends_with?(source, ".erl") ->
-        Process.put(:iex_reloaded, :ordsets.add_element(module, iex_reloaded))
-        [ compile_erlang(source) |> elem(0) ]
+        [compile_erlang(source) |> elem(0)]
 
       true ->
-        Process.put(:iex_reloaded, :ordsets.add_element(module, iex_reloaded))
         Enum.map(Code.load_file(source), fn {name, _} -> name end)
     end
   end
@@ -342,10 +331,6 @@ defmodule IEx.Helpers do
     end
   end
 
-  defp iex_reloaded do
-    Process.get(:iex_reloaded) || :ordsets.new
-  end
-
   defp source(module) do
     source = module.module_info(:compile)[:source]
 

lib/iex/test/iex/helpers_test.exs

@@ -266,20 +266,13 @@ defmodule IEx.HelpersTest do
     cleanup_modules([Sample])
   end
 
-  test "r helper basic" do
-    assert r == []
+  test "r helper unavailable" do
     assert_raise UndefinedFunctionError, "undefined function: :non_existent_module.module_info/1", fn ->
       r :non_existent_module
     end
-
-    # There is no source file for the module defined in IEx
-    assert ":ok\n** (Code.LoadError) could not load" <> _
-           = capture_iex("{:module, Sample, _, {:run,0}} = #{String.strip test_module_code}; :ok\nr Sample")
-  after
-    cleanup_modules([Sample])
   end
 
-  test "r helper" do
+  test "r helper elixir" do
     assert_raise UndefinedFunctionError, "undefined function: Sample.run/0", fn ->
       Sample.run
     end
@@ -296,8 +289,7 @@ defmodule IEx.HelpersTest do
           Sample.run
         end
 
-        assert [Sample] = r()
-      end) =~ %r"^.*?sample\.ex:1: redefining module Sample\n.*?sample\.ex:1: redefining module Sample\n$"
+      end) =~ %r"^.*?sample\.ex:1: redefining module Sample\n$"
     end
   after
     # Clean up old version produced by the r helper