Browse Source

dgram: scope redeclared variables

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: https://github.com/nodejs/node/pull/4940
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
v5.x
Rich Trott 9 years ago
committed by Rod Vagg
parent
commit
b4db31822f
  1. 8
      lib/dgram.js

8
lib/dgram.js

@ -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;
@ -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);
@ -371,7 +371,7 @@ function doSend(ex, self, ip, buffer, address, port, callback) {
!!callback);
if (err && callback) {
// don't emit as error, dgram_legacy.js compatibility
var ex = exceptionWithHostPort(err, 'send', address, port);
const ex = exceptionWithHostPort(err, 'send', address, port);
process.nextTick(callback, ex);
}
}

Loading…
Cancel
Save