dns: remove duplicate code

rickyes · targos · commit e0e73f6850dc · 2020-04-11T10:12:58.000+02:00
PR-URL: #32664 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: James M Snell <jasnell@gmail.com>
diff --git a/lib/dns.js b/lib/dns.js
@@ -65,11 +65,7 @@ function onlookup(err, addresses) {
   if (err) {
     return this.callback(dnsException(err, 'getaddrinfo', this.hostname));
   }
-  if (this.family) {
-    this.callback(null, addresses[0], this.family);
-  } else {
-    this.callback(null, addresses[0], isIP(addresses[0]));
-  }
+  this.callback(null, addresses[0], this.family || isIP(addresses[0]));
 }
 
 

Original file line number	Diff line number	Diff line change
`@@ -65,11 +65,7 @@ function onlookup(err, addresses) {`
`65`	`65`	`if (err) {`
`66`	`66`	`return this.callback(dnsException(err, 'getaddrinfo', this.hostname));`
`67`	`67`	`}`
`68`		`- if (this.family) {`
`69`		`- this.callback(null, addresses[0], this.family);`
`70`		`- } else {`
`71`		`- this.callback(null, addresses[0], isIP(addresses[0]));`
`72`		`- }`
	`68`	`+ this.callback(null, addresses[0], this.family \|\| isIP(addresses[0]));`
`73`	`69`	`}`
`74`	`70`
`75`	`71`