Skip to content

Commit

Permalink
net: replace __defineGetter__ with defineProperty
Browse files Browse the repository at this point in the history
`Object.prototype.__defineGetter__` is deprecated now, use
`Object.defineProperty` instead.

PR-URL: #6284
Reviewed-By: Ben Noordhuis <[email protected]>
  • Loading branch information
indutny authored and Myles Borins committed Apr 20, 2016
1 parent 20dcdd3 commit d5d4f19
Showing 1 changed file with 15 additions and 7 deletions.
22 changes: 15 additions & 7 deletions lib/net.js
Original file line number Diff line number Diff line change
Expand Up @@ -585,19 +585,27 @@ Socket.prototype._getpeername = function() {
return this._peername;
};

Socket.prototype.__defineGetter__('bytesRead', function() {
function protoGetter(name, callback) {
Object.defineProperty(Socket.prototype, name, {
configurable: false,
enumerable: true,
get: callback
});
}

protoGetter('bytesRead', function bytesRead() {
return this._handle ? this._handle.bytesRead : this[BYTES_READ];
});

Socket.prototype.__defineGetter__('remoteAddress', function() {
protoGetter('remoteAddress', function remoteAddress() {
return this._getpeername().address;
});

Socket.prototype.__defineGetter__('remoteFamily', function() {
protoGetter('remoteFamily', function remoteFamily() {
return this._getpeername().family;
});

Socket.prototype.__defineGetter__('remotePort', function() {
protoGetter('remotePort', function remotePort() {
return this._getpeername().port;
});

Expand All @@ -616,12 +624,12 @@ Socket.prototype._getsockname = function() {
};


Socket.prototype.__defineGetter__('localAddress', function() {
protoGetter('localAddress', function localAddress() {
return this._getsockname().address;
});


Socket.prototype.__defineGetter__('localPort', function() {
protoGetter('localPort', function localPort() {
return this._getsockname().port;
});

Expand Down Expand Up @@ -733,7 +741,7 @@ function createWriteReq(req, handle, data, encoding) {
}


Socket.prototype.__defineGetter__('bytesWritten', function() {
protoGetter('bytesWritten', function bytesWritten() {
var bytes = this._bytesDispatched;
const state = this._writableState;
const data = this._pendingData;
Expand Down

0 comments on commit d5d4f19

Please sign in to comment.