http/client: Track number of records separately

Author: daurnimator

http/client.lua

@@ -173,10 +173,13 @@ local function connect(options, timeout)
 
 	local host = options.host
 	local records
+	local n_records
 	if path then
 		records = { { family = family, path = path } }
+		n_records = 1
 	elseif http_util.is_ip(host) then
 		records = { { family = family, host = host } }
+		n_records = 1
 	else
 		local dns_resolver = options.dns_resolver or cqueues_dns.getpool()
 		records = {}
@@ -188,6 +191,7 @@ local function connect(options, timeout)
 		elseif family == cs.AF_INET6 then
 			dns_lookup(records, dns_resolver, host, cqueues_dns_record.AAAA, cqueues_dns_record.AAAA, timeout)
 		end
+		n_records = #records
 		timeout = deadline and deadline-monotime()
 	end
 
@@ -204,8 +208,7 @@ local function connect(options, timeout)
 
 	local lasterr, lasterrno = "The name does not resolve for the supplied parameters"
 	local i = 1
-	local n = #records
-	while i <= n do
+	while i <= n_records do
 		local rec = records[i]
 		connect_params.family = rec.family;
 		connect_params.host = rec.host;
@@ -232,14 +235,14 @@ local function connect(options, timeout)
 			-- If an address family is not supported then entirely remove that
 			-- family from candidate records
 			local af = connect_params.family
-			for j=n, i+1, -1 do
+			for j=n_records, i+1, -1 do
 				if records[j].family == af then
 					table.remove(records, j)
-					n = n - 1
+					n_records = n_records - 1
 				end
 			end
 			table.remove(records, i)
-			n = n - 1
+			n_records = n_records - 1
 		else
 			i = i + 1
 		end