Browse Source

udp: remove a needless instanceof Buffer check

When a string is passed to udpsock.send, it is automatically
converted to a Buffer. In that case, it is no longer needed
to test whether or not the argument is a Buffer or not.

PR-URL: https://github.com/nodejs/node/pull/4301
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Roman Reiss <me@silverwind.io>
v4.x
ronkorving 9 years ago
committed by Myles Borins
parent
commit
dd0c925896
  1. 3
      lib/dgram.js

3
lib/dgram.js

@ -252,8 +252,7 @@ Socket.prototype.send = function(buffer,
if (typeof buffer === 'string')
buffer = new Buffer(buffer);
if (!(buffer instanceof Buffer))
else if (!(buffer instanceof Buffer))
throw new TypeError('First argument must be a buffer or string.');
offset = offset | 0;

Loading…
Cancel
Save