[refactor] Simply the initialization logic for exception handling.

[breaking] No longer respect the `handleExceptions` option to `new winston.Logger`. Instead
just pass in the `exceptionHandlers` option itself.

Author: indexzero

lib/winston/logger.js

@@ -72,15 +72,7 @@ var Logger = exports.Logger = function (options) {
   }
 
   if (options.exceptionHandlers) {
-    handleExceptions = true;
-    options.exceptionHandlers.forEach(function (handler) {
-      self._hnames.push(handler.name);
-      self.exceptionHandlers[handler.name] = handler;
-    });
-  }
-
-  if (options.handleExceptions || handleExceptions) {
-    this.handleExceptions();
+    this.handleExceptions(options.exceptionHandlers);
   }
 };
 
@@ -377,8 +369,9 @@ Logger.prototype.close = function () {
 };
 
 //
-// ### function handleExceptions ()
-// Handles `uncaughtException` events for the current process
+// ### function handleExceptions ([tr0, tr1...] || tr0, tr1, ...)
+// Handles `uncaughtException` events for the current process by
+// ADDING any handlers passed in.
 //
 Logger.prototype.handleExceptions = function () {
   var args = Array.prototype.slice.call(arguments),
@@ -394,6 +387,7 @@ Logger.prototype.handleExceptions = function () {
     }
   });
 
+  this.exceptionHandlers = this.exceptionHandlers || {};
   handlers.forEach(function (handler) {
     self.exceptionHandlers[handler.name] = handler;
   });