diff mbox

[v2] sysdeps/posix: Simplify if expression in getaddrinfo()

Message ID 20191124193517.251297-1-tim.ruehsen@gmx.de
State Committed
Commit cceb038ac0ea552d5eb81759572a358179fa0fc1
Headers

Commit Message

Tim Rühsen Nov. 24, 2019, 7:35 p.m. UTC 
  Small code cleanup for better readability.
---
 sysdeps/posix/getaddrinfo.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

--
2.24.0

Comments

Florian Weimer Nov. 26, 2019, 9:18 a.m. UTC | #1 
* Tim Rühsen:

> Small code cleanup for better readability.
> ---
>  sysdeps/posix/getaddrinfo.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
> index ea127877cb..afdcdf0a3b 100644
> --- a/sysdeps/posix/getaddrinfo.c
> +++ b/sysdeps/posix/getaddrinfo.c
> @@ -2206,7 +2206,7 @@ getaddrinfo (const char *name, const char *service,
>  	{
>  	  /* If we haven't seen both IPv4 and IPv6 interfaces we can
>  	     narrow down the search.  */
> -	  if ((! seen_ipv4 || ! seen_ipv6) && (seen_ipv4 || seen_ipv6))
> +	  if (seen_ipv4 != seen_ipv6)
>  	    {
>  	      local_hints = *hints;
>  	      local_hints.ai_family = seen_ipv4 ? PF_INET : PF_INET6;

Thanks, I've pushed that for you, after dropping the () from the
subject (per GNU style).
diff mbox

Patch

diff --git a/sysdeps/posix/getaddrinfo.c b/sysdeps/posix/getaddrinfo.c
index ea127877cb..afdcdf0a3b 100644
--- a/sysdeps/posix/getaddrinfo.c
+++ b/sysdeps/posix/getaddrinfo.c
@@ -2206,7 +2206,7 @@  getaddrinfo (const char *name, const char *service,
 	{
 	  /* If we haven't seen both IPv4 and IPv6 interfaces we can
 	     narrow down the search.  */
-	  if ((! seen_ipv4 || ! seen_ipv6) && (seen_ipv4 || seen_ipv6))
+	  if (seen_ipv4 != seen_ipv6)
 	    {
 	      local_hints = *hints;
 	      local_hints.ai_family = seen_ipv4 ? PF_INET : PF_INET6;