From 9744d09e529f5fbb9bfca02e551f34dba29c1921 Mon Sep 17 00:00:00 2001
From: Ray Greenwell <ray@threerings.net>
Date: Tue, 6 Feb 2007 20:49:20 +0000
Subject: [PATCH] De-flexing: don't use the MX logging facilities. They're cool
 and allow filtering based on package, but we may not ever use that
 functionality and it's far more useful to be able to freely include this Log
 class without worrying that it will suck in some flex classes as well.

git-svn-id: svn+ssh://src.earth.threerings.net/narya/trunk@4535 542714f4-19e9-0310-aa3c-eee0fc999fb1
---
 src/as/Log.as | 45 ++++++++++++++++++---------------------------
 1 file changed, 18 insertions(+), 27 deletions(-)

diff --git a/src/as/Log.as b/src/as/Log.as
index 0b4aafa99..4424f9f3f 100644
--- a/src/as/Log.as
+++ b/src/as/Log.as
@@ -2,14 +2,13 @@ package {
 
 import flash.utils.getQualifiedClassName;
 
-import mx.logging.ILogger;
-import mx.logging.LogEventLevel;
+//import mx.logging.ILogger;
+//import mx.logging.LogEventLevel;
 
-import mx.logging.targets.TraceTarget;
+//import mx.logging.targets.TraceTarget;
 
 /**
- * A simple logging mechanism built on top of the standard mx logging
- * facilities.
+ * A simple logging mechanism.
  *
  * This class need not be imported.
  *
@@ -45,7 +44,7 @@ public class Log
     {
         // let's just use the full classname
         var path :String = getQualifiedClassName(spec).replace("::", ".");
-        return new Log(mx.logging.Log.getLogger(path));
+        return new Log(path);
     }
 
     /**
@@ -54,7 +53,7 @@ public class Log
      */
     public static function testing (... params) :void
     {
-        var log :ILogger = mx.logging.Log.getLogger("testing");
+        var log :Log = new Log("testing");
         log.debug.apply(log, params);
     }
 
@@ -70,9 +69,9 @@ public class Log
     /**
      * @private
      */
-    public function Log (dest :ILogger)
+    public function Log (spec :String)
     {
-        _dest = dest;
+        _spec = spec;
     }
 
     /**
@@ -80,7 +79,7 @@ public class Log
      */
     public function debug (... messages) :void
     {
-        _dest.debug.apply(_dest, messages);
+        doLog("[debug]", messages);
     }
 
     /**
@@ -88,7 +87,7 @@ public class Log
      */
     public function info (... messages) :void
     {
-        _dest.info.apply(_dest, messages);
+        doLog("[INFO]", messages);
     }
 
     /**
@@ -96,7 +95,7 @@ public class Log
      */
     public function warning (... messages) :void
     {
-        _dest.warn.apply(_dest, messages);
+        doLog("[WARNING]", messages);
     }
 
     /**
@@ -104,26 +103,18 @@ public class Log
      */
     public function logStackTrace (error :Error) :void
     {
-        _dest.warn(error.getStackTrace());
+        warning(error.getStackTrace());
     }
 
-    /**
-     * Our static (class) initializer.
-     */
-    private static function staticInit () :void
+    protected function doLog (level :String, messages :Array) :void
     {
-        var targ :TraceTarget = new TraceTarget();
-        targ.includeCategory = targ.includeDate = targ.includeLevel =
-                targ.includeTime = true;
-        targ.filters = ["*"]; // TODO
-        targ.level = LogEventLevel.DEBUG;
-        mx.logging.Log.addTarget(targ);
+        // TODO: better Date formatting?
+        messages.unshift(new Date().toLocaleTimeString(), level, _spec);
+        trace.apply(null, messages);
     }
 
-    staticInit(); // call the static initializer
-
-    /** Our true destination. */
-    protected var _dest :ILogger;
+    /** Our log specification. */
+    protected var _spec :String;
 }
 
 }