debug-fabulous 1.0.0 → 1.1.0

package/README.md

@@ -10,9 +10,9 @@ Wrapper / Extension around [visionmedia's debug](https://github.com/visionmedia/
 
 ## Why would I consider using this library?
 
-One on the main utilities added to this library is lazy log level evaluation. This allows whatever log strings to only be created and evaluated if a log level is active. This can considerably reduce the amount of memory used in logging when you are not using.
+The main utilities added to this library is lazy log level evaluation. This allows whatever logged strings to only be created and evaluated if a log level is active. This can considerably reduce the amount of memory used in logging when you are not using.
 
-This is important as this results in no-excuse for not using logging in your applications for performance reasons.
+With this in mind, there are no excuses to not log anything and everything as performance can be kept in check easily (via log levels).
 
 ### Proof
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "debug-fabulous",
-  "version": "1.0.0",
+  "version": "1.1.0",
   "description": "visionmedia debug extensions rolled into one",
   "main": "index.js",
   "scripts": {

package/src/lazy-eval.js

@@ -16,28 +16,24 @@ function _resolveOutput(func, bindThis) {
   objectAssign(wrapped, func);
 
   return wrapped;
-};
-
+}
 
 function wrapEval(_debug) {
-
   var debugOrig = _debug;
-  var noop = function(){};
 
   function debug(namespace) {
-
+    function noop() {}
     var instance = debugOrig(namespace);
-
-    // if we're not enabled then don't attempt to log anything
-    // if a debug namespace wraps its debug in a closure then it never allocates anything but the function itself
-    if (!instance.enabled){
+    /*
+      If we're not enabled then don't attempt to log anything.
+      Therefore when a  debug namespace wraps its debug in a
+      closure then it never allocates anything but the function itself
+    */
+    if (!instance.enabled) {
       objectAssign(noop, instance);
-      instance = noop;
-    }
-    else {
-      instance = _resolveOutput(instance);
+      return noop;
     }
-    return instance;
+    return _resolveOutput(instance);
   }
 
   var debugMemoized = memoize(debug);

package/src/spawn.js

@@ -1,5 +1,4 @@
 function spawnFactory(_namespace, _debugFabFactory) {
-  var memoize = require('memoizee');
   var namespace = _namespace || '';
   var debugFabFactory = _debugFabFactory;
 
@@ -7,6 +6,13 @@ function spawnFactory(_namespace, _debugFabFactory) {
     debugFabFactory = require('./debugFabFactory')();
   }
 
+  function spawn(ns) {
+    // this is this.debug (from Debugger)
+    var dbg = new Debugger(this.namespace, ns);
+
+    return dbg.debug;
+  };
+
   function Debugger(_base, _ns){
     var base = _base || '';
     var ns = _ns || '';
@@ -15,17 +21,9 @@ function spawnFactory(_namespace, _debugFabFactory) {
     var debug = debugFabFactory(newNs);
 
     this.debug = debug;
-    this.debug.spawn = this.spawn;
+    this.debug.spawn = spawn;
   }
 
-  Debugger.prototype.spawn = function(ns) {
-    var dbg = new Debugger(this.namespace, ns);
-
-    return dbg.debug;
-  };
-
-  Debugger.prototype.spawn = memoize(Debugger.prototype.spawn);
-
   var rootDebug = (new Debugger(namespace)).debug;
 
   return rootDebug;