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: nodejs#4301 Reviewed-By: Colin Ihrig <[email protected]> Reviewed-By: Brian White <[email protected]> Reviewed-By: Roman Reiss <[email protected]>
scovetta · Apr 2, 2016 · 8df5123 · 8df5123
1 parent dc02563
commit 8df5123
Showing 1 changed file with 1 addition and 2 deletions.
diff --git a/lib/dgram.js b/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;