Skip to content

Commit

Permalink
use Map instead of Object in loggers
Browse files Browse the repository at this point in the history
  • Loading branch information
Cap32 committed Oct 24, 2017
1 parent d7a0e53 commit bbbf339
Showing 1 changed file with 7 additions and 5 deletions.
12 changes: 7 additions & 5 deletions src/index.js
Original file line number Diff line number Diff line change
Expand Up @@ -43,7 +43,7 @@ const logSystem = (function () {
let enable = config.enable;
let appenders = {};
let categories = {};
const loggers = {};
const loggers = new Map();

const getCategories = () => {
const appenderKeys = Object.keys(appenders);
Expand Down Expand Up @@ -196,10 +196,10 @@ const logSystem = (function () {
shouldReload = false;
},
hasLogger(category) {
return !!loggers[category];
return loggers.has(category);
},
getLogger(category = defaultCategory) {
if (loggers[category]) { return loggers[category]; }
if (loggers.has(category)) { return loggers.get(category); }

let origin = null;
let cache = {};
Expand All @@ -223,15 +223,17 @@ const logSystem = (function () {
return (cache[name] = enable ? origin[name].bind(origin) : noop);
};

return (loggers[category] = {
const logger = {
[SymbolEnsureLatest]: ensureLatest,
get trace() { return reflect('trace'); },
get debug() { return reflect('debug'); },
get info() { return reflect('info'); },
get warn() { return reflect('warn'); },
get error() { return reflect('error'); },
get fatal() { return reflect('fatal'); },
});
};
loggers.set(category, logger);
return logger;
},
};
}());
Expand Down

0 comments on commit bbbf339

Please sign in to comment.