Cleaning up logger

Author: jwarwick

lib/elixir_osc.ex

@@ -6,8 +6,4 @@ defmodule ElixirOsc do
   def start(_type, args) do
     ElixirOsc.Supervisor.start_link args
   end
-
-  def start_logger do
-    ElixirOsc.Events.subscribe(ElixirOsc.Logger, [])
-  end
 end

lib/elixir_osc/events.ex

@@ -6,8 +6,14 @@ defmodule ElixirOsc.Events do
     {:ok, []}
   end
 
-  def subscribe(module, args) do
-    :gen_event.add_handler(:osc_events, module, args)
+  def subscribe(handler), do: subscribe(handler, [])
+  def subscribe(handler, args) do
+    :gen_event.add_handler(:osc_events, handler, args)
+  end
+
+  def unsubscribe(handler), do: unsubscribe(handler, [])
+  def unsubscribe(handler, args) do
+    :gen_event.delete_handler(:osc_events, handler, args)
   end
 
   def send_event(msg) do

lib/elixir_osc/logger.ex

@@ -1,6 +1,16 @@
 defmodule ElixirOsc.Logger do
   use GenEvent.Behaviour
 
+  def start_logger do
+    pid = {ElixirOsc.Logger, make_ref}
+    :ok = ElixirOsc.Events.subscribe(pid)
+    {:ok, pid}
+  end
+
+  def stop_logger(pid) do
+    ElixirOsc.Events.unsubscribe(pid) 
+  end
+
   def handle_event(event, state) do
     IO.inspect event
     {:ok, state}