Skip to content

Commit

Permalink
dgram: scope redeclared variables
Browse files Browse the repository at this point in the history
A few variables in `lib/dgram.js` are redeclared with `var` in a scope
where they have already been declared. These instances can be scoped
more narrowly with `const`, so that's what this change does.

PR-URL: #4940
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: James M Snell <[email protected]>
  • Loading branch information
Trott authored and Myles Borins committed Feb 18, 2016
1 parent 43d4db4 commit 82f549e
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions lib/dgram.js
Original file line number Diff line number Diff line change
Expand Up @@ -40,13 +40,13 @@ function lookup6(address, callback) {

function newHandle(type) {
if (type == 'udp4') {
var handle = new UDP();
const handle = new UDP();
handle.lookup = lookup4;
return handle;
}

if (type == 'udp6') {
var handle = new UDP();
const handle = new UDP();
handle.lookup = lookup6;
handle.bind = handle.bind6;
handle.send = handle.send6;
Expand Down Expand Up @@ -209,7 +209,7 @@ Socket.prototype.bind = function(port_ /*, address, callback*/) {
if (!self._handle)
return; // handle has been closed in the mean time

var err = self._handle.bind(ip, port || 0, flags);
const err = self._handle.bind(ip, port || 0, flags);
if (err) {
var ex = exceptionWithHostPort(err, 'bind', ip, port);
self.emit('error', ex);
Expand Down

0 comments on commit 82f549e

Please sign in to comment.