refactor: logger moved away from ircbot (exp.)

This moves the logger management thread/queue to a seperate singleton.
It removes the explicit stopping/restarting of the logging thread since
the thread should behave like a daemon thread anyway.
Still needs to be tested to work in daemonize.

1 files changed, 7 insertions, 4 deletions

diff --git a/bin/rbot b/bin/rbot
index a8dd39c4..86f907cf 100755
--- a/bin/rbot
+++ b/bin/rbot
@@ -61,17 +61,13 @@ opts = GetoptLong.new(
   ["--version", "-v", GetoptLong::NO_ARGUMENT]
 )
 
-$debug = $DEBUG
 $daemonize = false
 
 opts.each {|opt, arg|
-  $debug = true if(opt == "--debug")
   $daemonize = true if(opt == "--background")
   $opts[opt.sub(/^-+/, "")] = arg
 }
 
-$cl_loglevel = $opts["loglevel"].to_i if $opts["loglevel"]
-
 if ($opts["trace"])
   set_trace_func proc { |event, file, line, id, binding, classname|
     if classname.to_s == $opts["trace"]
@@ -116,6 +112,13 @@ if ($opts["help"])
 end
 
 if(bot = Irc::Bot.new(ARGV.shift, :argv => orig_opts))
+  # setup logger based on command line arguments
+  loglevel = $opts['loglevel'] ? $opts['loglevel'].to_i : nil
+  loglevel = $opts['debug'] ? 0 : nil
+  if loglevel
+    Irc::Bot::LoggerManager.instance.set_level(loglevel)
+  end
+
   # just run the bot
   bot.mainloop
 end