lib,src: remove usage of events.EventEmitter
authorSakthipriyan Vairamani <thechargingvolcano@gmail.com>
Wed, 16 Sep 2015 22:45:29 +0000 (04:15 +0530)
committerRod Vagg <rod@vagg.org>
Tue, 22 Sep 2015 22:39:34 +0000 (08:39 +1000)
The `events` module already exports `EventEmitter` constructor function
So, we don't have to use `events.EventEmitter` to access it.

Refer: https://github.com/nodejs/node/pull/2896

PR-URL: https://github.com/nodejs/node/pull/2921
Reviewed-By: Roman Reiss <me@silverwind.io>
Reviewed-By: Michaƫl Zasso <mic.besace@gmail.com>
15 files changed:
lib/_http_agent.js
lib/_http_client.js
lib/_http_server.js
lib/_stream_readable.js
lib/_tls_legacy.js
lib/cluster.js
lib/dgram.js
lib/fs.js
lib/http.js
lib/internal/child_process.js
lib/internal/socket_list.js
lib/net.js
lib/readline.js
lib/stream.js
src/node.js

index c18f39a..305baa2 100644 (file)
@@ -2,7 +2,7 @@
 
 const net = require('net');
 const util = require('util');
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const debug = util.debuglog('http');
 
 // New Agent code.
index 8ddf334..aef05eb 100644 (file)
@@ -3,7 +3,7 @@
 const util = require('util');
 const net = require('net');
 const url = require('url');
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const HTTPParser = process.binding('http_parser').HTTPParser;
 const assert = require('assert').ok;
 const common = require('_http_common');
index 7acc108..f2e2dcf 100644 (file)
@@ -2,7 +2,7 @@
 
 const util = require('util');
 const net = require('net');
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const HTTPParser = process.binding('http_parser').HTTPParser;
 const assert = require('assert').ok;
 const common = require('_http_common');
@@ -117,7 +117,8 @@ function onServerResponseClose() {
   // array. That is, in the example below, b still gets called even though
   // it's been removed by a:
   //
-  //   var obj = new events.EventEmitter;
+  //   var EventEmitter = require('events');
+  //   var obj = new EventEmitter();
   //   obj.on('event', a);
   //   obj.on('event', b);
   //   function a() { obj.removeListener('event', b) }
index c6bd0c1..64ff744 100644 (file)
@@ -3,7 +3,7 @@
 module.exports = Readable;
 Readable.ReadableState = ReadableState;
 
-const EE = require('events').EventEmitter;
+const EE = require('events');
 const Stream = require('stream');
 const Buffer = require('buffer').Buffer;
 const util = require('util');
index e0c60db..1d700c9 100644 (file)
@@ -1,7 +1,7 @@
 'use strict';
 
 const assert = require('assert');
-const events = require('events');
+const EventEmitter = require('events');
 const stream = require('stream');
 const tls = require('tls');
 const util = require('util');
@@ -672,7 +672,7 @@ function SecurePair(context, isServer, requestCert, rejectUnauthorized,
 
   options || (options = {});
 
-  events.EventEmitter.call(this);
+  EventEmitter.call(this);
 
   this.server = options.server;
   this._secureEstablished = false;
@@ -741,7 +741,7 @@ function SecurePair(context, isServer, requestCert, rejectUnauthorized,
   process.nextTick(securePairNT, this, options);
 }
 
-util.inherits(SecurePair, events.EventEmitter);
+util.inherits(SecurePair, EventEmitter);
 
 function securePairNT(self, options) {
   /* The Connection may be destroyed by an abort call */
index 84d7c76..602cc8d 100644 (file)
@@ -1,6 +1,6 @@
 'use strict';
 
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const assert = require('assert');
 const dgram = require('dgram');
 const fork = require('child_process').fork;
index 5d207a5..eaf8427 100644 (file)
@@ -3,7 +3,7 @@
 const assert = require('assert');
 const Buffer = require('buffer').Buffer;
 const util = require('util');
-const events = require('events');
+const EventEmitter = require('events');
 const constants = require('constants');
 
 const UDP = process.binding('udp_wrap').UDP;
@@ -79,7 +79,7 @@ exports._createSocketHandle = function(address, port, addressType, fd, flags) {
 
 
 function Socket(type, listener) {
-  events.EventEmitter.call(this);
+  EventEmitter.call(this);
 
   if (typeof type === 'object') {
     var options = type;
@@ -101,7 +101,7 @@ function Socket(type, listener) {
   if (typeof listener === 'function')
     this.on('message', listener);
 }
-util.inherits(Socket, events.EventEmitter);
+util.inherits(Socket, EventEmitter);
 exports.Socket = Socket;
 
 
index eeb3806..76f53db 100644 (file)
--- a/lib/fs.js
+++ b/lib/fs.js
@@ -12,7 +12,7 @@ const constants = require('constants');
 const fs = exports;
 const Buffer = require('buffer').Buffer;
 const Stream = require('stream').Stream;
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const FSReqWrap = binding.FSReqWrap;
 const FSEvent = process.binding('fs_event_wrap').FSEvent;
 
index cead865..64788dd 100644 (file)
@@ -2,7 +2,7 @@
 
 const util = require('util');
 const internalUtil = require('internal/util');
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 
 
 exports.IncomingMessage = require('_http_incoming').IncomingMessage;
index 328a67d..d159640 100644 (file)
@@ -2,7 +2,7 @@
 
 const StringDecoder = require('string_decoder').StringDecoder;
 const Buffer = require('buffer').Buffer;
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const net = require('net');
 const dgram = require('dgram');
 const util = require('util');
index e9b388d..950d632 100644 (file)
@@ -2,7 +2,7 @@
 
 module.exports = {SocketListSend, SocketListReceive};
 
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 const util = require('util');
 
 // This object keep track of the socket there are sended
index 8245601..47422c3 100644 (file)
@@ -1,6 +1,6 @@
 'use strict';
 
-const events = require('events');
+const EventEmitter = require('events');
 const stream = require('stream');
 const timers = require('timers');
 const util = require('util');
@@ -1071,7 +1071,7 @@ function Server(options, connectionListener) {
   if (!(this instanceof Server))
     return new Server(options, connectionListener);
 
-  events.EventEmitter.call(this);
+  EventEmitter.call(this);
 
   var self = this;
   var options;
@@ -1113,7 +1113,7 @@ function Server(options, connectionListener) {
   this.allowHalfOpen = options.allowHalfOpen || false;
   this.pauseOnConnect = !!options.pauseOnConnect;
 }
-util.inherits(Server, events.EventEmitter);
+util.inherits(Server, EventEmitter);
 exports.Server = Server;
 
 
index 0bdf5c1..0f9aabb 100644 (file)
@@ -12,7 +12,7 @@ const util = require('util');
 const internalUtil = require('internal/util');
 const inherits = util.inherits;
 const Buffer = require('buffer').Buffer;
-const EventEmitter = require('events').EventEmitter;
+const EventEmitter = require('events');
 
 
 exports.createInterface = function(input, output, completer, terminal) {
index 8d3535d..047fbce 100644 (file)
@@ -2,7 +2,7 @@
 
 module.exports = Stream;
 
-const EE = require('events').EventEmitter;
+const EE = require('events');
 const util = require('util');
 
 util.inherits(Stream, EE);
index bd91fb8..001a476 100644 (file)
@@ -10,7 +10,7 @@
   this.global = this;
 
   function startup() {
-    var EventEmitter = NativeModule.require('events').EventEmitter;
+    var EventEmitter = NativeModule.require('events');
 
     process.__proto__ = Object.create(EventEmitter.prototype, {
       constructor: {