Browse Source

net: remove hot path comment from connect

This comment was added with an assumption that we could determine the
IP address that localhost should resolve to without performing a
lookup. This was a false assumption and should be removed.

PR-URL: https://github.com/nodejs/node/pull/4648
Reviewed-By: Brian White <mscdex@mscdex.net>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
v4.x
Evan Lucas 9 years ago
committed by Myles Borins
parent
commit
40c8e6d75d
  1. 1
      lib/net.js

1
lib/net.js

@ -931,7 +931,6 @@ function lookupAndConnect(self, options) {
port |= 0;
// If host is an IP, skip performing a lookup
// TODO(evanlucas) should we hot path this for localhost?
var addressType = exports.isIP(host);
if (addressType) {
process.nextTick(function() {

Loading…
Cancel
Save