lib: reduce usage of `self = this`

Remove unnecessary `self = this`.

PR-URL: https://github.com/nodejs/node/pull/5231
Reviewed-By: James M Snell <jasnell@gmail.com>

1 files changed, 3 insertions, 4 deletions

diff --git a/lib/domain.js b/lib/domain.js
index 41b4859361..d6958b2c30 100644
--- a/lib/domain.js
+++ b/lib/domain.js
@@ -60,7 +60,6 @@ Domain.prototype._disposed = undefined;
 // Called by process._fatalException in case an error was thrown.
 Domain.prototype._errorHandler = function errorHandler(er) {
   var caught = false;
-  var self = this;
 
   // ignore errors on disposed domains.
   //
@@ -89,13 +88,13 @@ Domain.prototype._errorHandler = function errorHandler(er) {
     // as this would throw an error, make the process exit, and thus
     // prevent the process 'uncaughtException' event from being emitted
     // if a listener is set.
-    if (EventEmitter.listenerCount(self, 'error') > 0) {
+    if (EventEmitter.listenerCount(this, 'error') > 0) {
       try {
         // Set the _emittingTopLevelDomainError so that we know that, even
         // if technically the top-level domain is still active, it would
         // be ok to abort on an uncaught exception at this point
         process._emittingTopLevelDomainError = true;
-        caught = self.emit('error', er);
+        caught = this.emit('error', er);
       } finally {
         process._emittingTopLevelDomainError = false;
       }
@@ -111,7 +110,7 @@ Domain.prototype._errorHandler = function errorHandler(er) {
       //
       // If caught is false after this, then there's no need to exit()
       // the domain, because we're going to crash the process anyway.
-      caught = self.emit('error', er);
+      caught = this.emit('error', er);
     } catch (er2) {
       // The domain error handler threw!  oh no!
       // See if another domain can catch THIS error,