Move from already_added to already_present

Author: José Valim

lib/elixir/lib/gen_event.ex

@@ -315,7 +315,7 @@ defmodule GenEvent do
   the event handler is ignored and this function returns `{:error, reason}`.
 
   If the given handler was previously installed at the manager, this
-  function returns `{:error, :already_added}`.
+  function returns `{:error, :already_present}`.
 
   ## Monitored handlers
 
@@ -960,7 +960,7 @@ defmodule GenEvent do
             {false, error, handlers}
         end
       _ ->
-        {false, {:error, :already_added}, handlers}
+        {false, {:error, :already_present}, handlers}
     end
   end
 
@@ -979,7 +979,7 @@ defmodule GenEvent do
           server_add_handler(module2, {args2, state}, [])
         end
       else
-        {false, {:error, :already_added}, []}
+        {false, {:error, :already_present}, []}
       end
 
     case res do

lib/elixir/test/elixir/gen_event_test.exs

@@ -234,7 +234,7 @@ defmodule GenEventTest do
 
     assert GenEvent.add_handler(pid, ReplyHandler, {:throw, self()}) == :ok
     assert GenEvent.which_handlers(pid) == [ReplyHandler]
-    assert GenEvent.add_handler(pid, ReplyHandler, {:throw, self()}) == {:error, :already_added}
+    assert GenEvent.add_handler(pid, ReplyHandler, {:throw, self()}) == {:error, :already_present}
 
     assert GenEvent.add_handler(pid, {ReplyHandler, self()}, {self(), false}) == :ok
     assert GenEvent.which_handlers(pid) == [{ReplyHandler, self()}, ReplyHandler]
@@ -317,7 +317,7 @@ defmodule GenEventTest do
 
     assert GenEvent.add_handler(pid, ReplyHandler, {self(), false}) == :ok
     assert GenEvent.swap_handler(pid, ReplyHandler, :swapped,
-                                 {ReplyHandler, self()}, :swap) == {:error, :already_added}
+                                 {ReplyHandler, self()}, :swap) == {:error, :already_present}
     assert GenEvent.which_handlers(pid) == [{ReplyHandler, self()}]
 
     assert GenEvent.remove_handler(pid, {ReplyHandler, self()}, :remove_handler) ==

lib/logger/test/logger_test.exs

@@ -42,7 +42,7 @@ defmodule LoggerTest do
 
     assert {:ok, _pid} = Logger.add_backend(:console)
     assert Application.get_env(:logger, :backends) == [:console]
-    assert Logger.add_backend(:console) == {:error, :already_added}
+    assert Logger.add_backend(:console) == {:error, :already_present}
     assert Application.get_env(:logger, :backends) == [:console]
   end
 
@@ -56,7 +56,7 @@ defmodule LoggerTest do
     end
 
     assert {:ok, _} = Logger.add_backend({MyBackend, :hello})
-    assert {:error, :already_added} = Logger.add_backend({MyBackend, :hello})
+    assert {:error, :already_present} = Logger.add_backend({MyBackend, :hello})
     assert :ok = Logger.remove_backend({MyBackend, :hello})
   end
 
@@ -195,7 +195,7 @@ defmodule LoggerTest do
 
     assert {:ok, pid} = Logger.add_backend(:console)
     assert Logger.add_backend(:console) ==
-           {:error, :already_added}
+           {:error, :already_present}
   after
     Application.put_env(:logger, :backends, [:console])
     Logger.App.stop()