http: simplify code and remove unused properties

nodejs · May 1, 2015 · ebe5d52 · ebe5d52
1 parent 0450ce7
commit ebe5d52
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 10 deletions.
diff --git a/lib/_http_common.js b/lib/_http_common.js
@@ -126,12 +126,6 @@ function parserOnMessageComplete() {
       parser._url = '';
     }
 
-    if (!stream.upgrade)
-      // For upgraded connections, also emit this after parser.execute
-      stream.push(null);
-  }
-
-  if (stream && !parser.incoming._pendings.length) {
     // For emit end event
     stream.push(null);
   }

diff --git a/lib/_http_incoming.js b/lib/_http_incoming.js
@@ -38,8 +38,6 @@ function IncomingMessage(socket) {
 
   this.readable = true;
 
-  this._pendings = [];
-  this._pendingIndex = 0;
   this.upgrade = null;
 
   // request (server) only
@@ -49,7 +47,6 @@ function IncomingMessage(socket) {
   // response (client) only
   this.statusCode = null;
   this.statusMessage = null;
-  this.client = this.socket;
 
   // flag for backwards compatibility grossness.
   this._consuming = false;
@@ -63,6 +60,16 @@ util.inherits(IncomingMessage, Stream.Readable);
 
 exports.IncomingMessage = IncomingMessage;
 
+Object.defineProperty(IncomingMessage.prototype, 'client', {
+  configurable: true,
+  enumerable: true,
+  get: util.deprecate(function() {
+    return this.socket;
+  }, 'client is deprecated, use socket or connection instead'),
+  set: util.deprecate(function(val) {
+    this.socket = this.connection = val;
+  }, 'client is deprecated, use socket or connection instead')
+});
 
 IncomingMessage.prototype.setTimeout = function(msecs, callback) {
   if (callback)

diff --git a/lib/_http_outgoing.js b/lib/_http_outgoing.js
@@ -62,7 +62,6 @@ function OutgoingMessage() {
   this._trailer = '';
 
   this.finished = false;
-  this._hangupClose = false;
   this._headerSent = false;
 
   this.socket = null;