From 40c8e6d75d85ff3f8d491146f330aa9cd68d0c92 Mon Sep 17 00:00:00 2001 From: Evan Lucas Date: Tue, 12 Jan 2016 06:32:04 -0600 Subject: [PATCH] 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 Reviewed-By: Colin Ihrig --- lib/net.js | 1 - 1 file changed, 1 deletion(-) diff --git a/lib/net.js b/lib/net.js index c5a83b0..21f1f8e 100644 --- a/lib/net.js +++ b/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() { -- 2.7.4