fix flake8 issues, clarify comment

This commit is contained in:
Scott Kuhl 2020-10-17 14:43:09 -04:00
parent 036c49e412
commit 64d5c77a71
2 changed files with 9 additions and 8 deletions

View File

@ -48,10 +48,11 @@ def main():
return hostwatch.hw_main(opt.subnets, opt.auto_hosts) return hostwatch.hw_main(opt.subnets, opt.auto_hosts)
else: else:
# parse_subnetports() is used to create a list of includes # parse_subnetports() is used to create a list of includes
# and excludes. It returns a list of one or more items for # and excludes. It is called once for each parameter and
# each subnet. It returns a list since "-x example.com" # returns a list of one or more items for each subnet (it
# might find that example.com has multiple IP addresses. Here, # can return more than one item when a hostname in the
# we flatten these lists. # parameter resolves to multiple IP addresses. Here, we
# flatten these lists.
includes = [item for sublist in opt.subnets+opt.subnets_file includes = [item for sublist in opt.subnets+opt.subnets_file
for item in sublist] for item in sublist]
excludes = [item for sublist in opt.exclude for item in sublist] excludes = [item for sublist in opt.exclude for item in sublist]

View File

@ -94,7 +94,6 @@ def parse_subnetport(s):
rv.append((family, addr[0], cidr_to_use, rv.append((family, addr[0], cidr_to_use,
int(fport or 0), int(lport or fport or 0))) int(fport or 0), int(lport or fport or 0)))
return rv return rv
@ -124,7 +123,8 @@ def parse_ipport(s):
raise Fatal('Unable to resolve address: %s' % host) raise Fatal('Unable to resolve address: %s' % host)
if len(addrinfo) > 1: if len(addrinfo) > 1:
print("WARNING: Host %s has more than one IP, only using one of them." % host) print("WARNING: Host %s has more than one IP, only using one of them."
% host)
family, _, _, _, addr = min(addrinfo) family, _, _, _, addr = min(addrinfo)
# Note: addr contains (ip, port) # Note: addr contains (ip, port)