A couple quick cleanups for LoggerFactory

lib/logging/logger_factory.js

@@ -21,24 +21,29 @@
 
 var ModuleLogger = require('./module_logger.js');
 
+// TODO This mapping should eventually go away. It seems that log
+// level names by convention (logger name + "LogLevel") is the way
+// to go.
+var loggerNamesToLevels = {
+    damping: 'dampingLogLevel',
+    dissemination: 'disseminationLogLevel',
+    gossip: 'gossipLogLevel',
+    join: 'joinLogLevel',
+    membership: 'membershipLogLevel',
+    suspicion: 'suspicionLogLevel',
+};
+
 function LoggerFactory(opts) {
     this.ringpop = opts.ringpop;
-    this.defaultLogger = new ModuleLogger(this.ringpop, 'defaultLogLevel');
     this.loggers = {}; // indexed by name
-    // TODO This mapping should eventually go away. It seems that log
-    // level names by convention (logger name + "LogLevel") is the way
-    // to go.
-    this.loggerNamesToLevels = {
-        damping: 'dampingLogLevel',
-        dissemination: 'disseminationLogLevel',
-        gossip: 'gossipLogLevel',
-        join: 'joinLogLevel',
-        membership: 'membershipLogLevel',
-        suspicion: 'suspicionLogLevel',
-    };
+    this.defaultLogger = new ModuleLogger(this.ringpop, 'defaultLogLevel');
 }
 
 LoggerFactory.prototype.getLogger = function getLogger(name) {
+    if (!name) {
+        return this.defaultLogger;
+    }
+
     if (this.loggers[name]) {
         return this.loggers[name];
     }
@@ -47,9 +52,10 @@ LoggerFactory.prototype.getLogger = function getLogger(name) {
 };
 
 LoggerFactory.prototype._createLogger = function _createLogger(name) {
-    var level = this.loggerNamesToLevels[name] || 'defaultLogLevel';
+    var level = loggerNamesToLevels[name];
+    if (!level) return this.defaultLogger;
     this.loggers[name] = new ModuleLogger(this.ringpop, level);
     return this.loggers[name];
-}
+};
 
 module.exports = LoggerFactory;