From 5775c2a87e53a8214be1a57c50cb3506166b4f83 Mon Sep 17 00:00:00 2001 From: Andre Noll Date: Sun, 8 Sep 2013 05:16:46 +0000 Subject: [PATCH] net: Reduce indentation level in makesock_addrinfo(). Handle the active socket case of makesock_addrinfo() first, i.e. transform if (a) lots_of_stuff; else short; into if (!a) { short; continue; } lots_of_stuff; --- net.c | 41 ++++++++++++++++++++--------------------- 1 file changed, 20 insertions(+), 21 deletions(-) diff --git a/net.c b/net.c index 28d11d66..b7806a18 100644 --- a/net.c +++ b/net.c @@ -430,31 +430,30 @@ static int makesock_addrinfo(unsigned l4type, bool passive, struct addrinfo *ai, continue; sockfd = ret; flowopt_setopts(sockfd, fo); - if (passive) { - /* - * Reuse the address on passive sockets to avoid - * failure on restart (protocols using listen()) and - * when creating multiple listener instances (UDP - * multicast). - */ - if (setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, &on, - sizeof(on)) == -1) { - ret = -ERRNO_TO_PARA_ERROR(errno); - close(sockfd); - PARA_ERROR_LOG("can not set SO_REUSEADDR: %s\n", - para_strerror(-ret)); - return ret; - } - if (bind(sockfd, ai->ai_addr, ai->ai_addrlen) < 0) { - close(sockfd); - continue; - } - return sockfd; - } else { + if (!passive) { if (connect(sockfd, ai->ai_addr, ai->ai_addrlen) == 0) return sockfd; close(sockfd); + continue; + } + /* + * Reuse the address on passive sockets to avoid failure on + * restart (protocols using listen()) and when creating + * multiple listener instances (UDP multicast). + */ + if (setsockopt(sockfd, SOL_SOCKET, SO_REUSEADDR, &on, + sizeof(on)) == -1) { + ret = -ERRNO_TO_PARA_ERROR(errno); + close(sockfd); + PARA_ERROR_LOG("can not set SO_REUSEADDR: %s\n", + para_strerror(-ret)); + return ret; + } + if (bind(sockfd, ai->ai_addr, ai->ai_addrlen) < 0) { + close(sockfd); + continue; } + return sockfd; } return -E_MAKESOCK; } -- 2.39.5