C++: double underscores are reserved
[awesomized/libmemcached] / src / libmemcached / connect.cc
index 7579de5c8da42aa12e9d9ad905a038c8488b24de..84617133741a9e5332e212ac50f29af4f2a8b563 100644 (file)
@@ -1,93 +1,46 @@
-/*  vim:expandtab:shiftwidth=2:tabstop=2:smarttab:
- *
- *  Libmemcached library
- *
- *  Copyright (C) 2011 Data Differential, http://datadifferential.com/
- *  Copyright (C) 2006-2010 Brian Aker All rights reserved.
- *
- *  Redistribution and use in source and binary forms, with or without
- *  modification, are permitted provided that the following conditions are
- *  met:
- *
- *      * Redistributions of source code must retain the above copyright
- *  notice, this list of conditions and the following disclaimer.
- *
- *      * Redistributions in binary form must reproduce the above
- *  copyright notice, this list of conditions and the following disclaimer
- *  in the documentation and/or other materials provided with the
- *  distribution.
- *
- *      * The names of its contributors may not be used to endorse or
- *  promote products derived from this software without specific prior
- *  written permission.
- *
- *  THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS
- *  "AS IS" AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT
- *  LIMITED TO, THE IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR
- *  A PARTICULAR PURPOSE ARE DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT
- *  OWNER OR CONTRIBUTORS BE LIABLE FOR ANY DIRECT, INDIRECT, INCIDENTAL,
- *  SPECIAL, EXEMPLARY, OR CONSEQUENTIAL DAMAGES (INCLUDING, BUT NOT
- *  LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR SERVICES; LOSS OF USE,
- *  DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER CAUSED AND ON ANY
- *  THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY, OR TORT
- *  (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
- *  OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
- *
- */
-
+/*
+    +--------------------------------------------------------------------+
+    | libmemcached - C/C++ Client Library for memcached                  |
+    +--------------------------------------------------------------------+
+    | Redistribution and use in source and binary forms, with or without |
+    | modification, are permitted under the terms of the BSD license.    |
+    | You should have received a copy of the license in a bundled file   |
+    | named LICENSE; in case you did not receive a copy you can review   |
+    | the terms online at: https://opensource.org/licenses/BSD-3-Clause  |
+    +--------------------------------------------------------------------+
+    | Copyright (c) 2006-2014 Brian Aker   https://datadifferential.com/ |
+    | Copyright (c) 2020 Michael Wallner   <mike@php.net>                |
+    +--------------------------------------------------------------------+
+*/
 
 #include "libmemcached/common.h"
+#include "p9y/poll.hpp"
 
 #include <cassert>
 
-#ifndef SOCK_CLOEXEC
-#  define SOCK_CLOEXEC 0
-#endif
-
-#ifndef SOCK_NONBLOCK
-# define SOCK_NONBLOCK 0
-#endif
-
-#ifndef FD_CLOEXEC
-# define FD_CLOEXEC 0
-#endif
 
-#ifndef SO_NOSIGPIPE
-# define SO_NOSIGPIPE 0
-#endif
-
-#ifndef TCP_NODELAY
-# define TCP_NODELAY 0
-#endif
-
-#ifndef TCP_KEEPIDLE
-# define TCP_KEEPIDLE 0
-#endif
-
-static memcached_return_t connect_poll(memcached_instance_st* server, const int connection_error)
-{
+static memcached_return_t connect_poll(memcached_instance_st *server, const int connection_error) {
   struct pollfd fds[1];
-  fds[0].fd= server->fd;
-  fds[0].events= server->events();
-  fds[0].revents= 0;
+  fds[0].fd = server->fd;
+  fds[0].events = server->events();
+  fds[0].revents = 0;
 
-  size_t loop_max= 5;
+  size_t loop_max = 5;
 
-  if (server->root->poll_timeout == 0)
-  {
-    return memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
-                               memcached_literal_param("The time to wait for a connection to be established was set to zero which produces a timeout to every call to poll()."));
+  if (server->root->connect_timeout == 0) {
+    return memcached_set_error(
+        *server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
+        memcached_literal_param("The time to wait for a connection to be established was set to "
+                                "zero which produces a timeout to every call to poll()."));
   }
 
   while (--loop_max) // Should only loop on cases of ERESTART or EINTR
   {
     int number_of;
-    if ((number_of= poll(fds, 1, server->root->connect_timeout)) == -1)
-    {
-      int local_errno= get_socket_errno(); // We cache in case closesocket() modifies errno
-      switch (local_errno)
-      {
-#ifdef __linux__
+    if ((number_of = poll(fds, 1, server->root->connect_timeout)) == SOCKET_ERROR) {
+      int local_errno = get_socket_errno(); // We cache in case closesocket() modifies errno
+      switch (local_errno) {
+#ifdef HAVE_ERESTART
       case ERESTART:
 #endif
       case EINTR:
@@ -98,8 +51,10 @@ static memcached_return_t connect_poll(memcached_instance_st* server, const int
         return memcached_set_error(*server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT);
 
       case EINVAL:
-        return memcached_set_error(*server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT,
-                                   memcached_literal_param("RLIMIT_NOFILE exceeded, or if OSX the timeout value was invalid"));
+        return memcached_set_error(
+            *server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT,
+            memcached_literal_param(
+                "RLIMIT_NOFILE exceeded, or if OSX the timeout value was invalid"));
 
       default: // This should not happen
         break;
@@ -107,43 +62,45 @@ static memcached_return_t connect_poll(memcached_instance_st* server, const int
 
       assert_msg(server->fd != INVALID_SOCKET, "poll() was passed an invalid file descriptor");
       server->reset_socket();
-      server->state= MEMCACHED_SERVER_STATE_NEW;
+      server->state = MEMCACHED_SERVER_STATE_NEW;
 
       return memcached_set_errno(*server, local_errno, MEMCACHED_AT);
     }
 
-    if (number_of == 0)
-    {
-      if (connection_error == EINPROGRESS)
-      {
+    if (number_of == 0) {
+      if (connection_error != EALREADY) {
         int err;
-        socklen_t len= sizeof(err);
-        if (getsockopt(server->fd, SOL_SOCKET, SO_ERROR, (char*)&err, &len) == -1)
-        {
-          return memcached_set_errno(*server, errno, MEMCACHED_AT, memcached_literal_param("getsockopt() error'ed while looking for error connect_poll(EINPROGRESS)"));
+        socklen_t len = sizeof(err);
+        if (getsockopt(server->fd, SOL_SOCKET, SO_ERROR, (char *) &err, &len) == -1) {
+          return memcached_set_errno(
+              *server, errno, MEMCACHED_AT,
+              memcached_literal_param(
+                  "getsockopt() error'ed while looking for error connect_poll(EINPROGRESS)"));
         }
 
         // If Zero, my hero, we just fail to a generic MEMCACHED_TIMEOUT error
-        if (err != 0)
-        {
-          return memcached_set_errno(*server, err, MEMCACHED_AT, memcached_literal_param("getsockopt() found the error from poll() after connect() returned EINPROGRESS."));
+        if (err) {
+          return memcached_set_errno(
+              *server, err, MEMCACHED_AT,
+              memcached_literal_param("getsockopt() found the error from poll() after connect() "
+                                      "returned EINPROGRESS."));
         }
       }
 
-      return  memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT, memcached_literal_param("(number_of == 0)"));
+      return memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
+                                 memcached_literal_param("(number_of == 0)"));
     }
 
-    assert (number_of == 1);
+    assert(number_of == 1);
 
-    if (fds[0].revents & POLLERR or
-        fds[0].revents & POLLHUP or
-        fds[0].revents & POLLNVAL)
-    {
+    if (fds[0].revents & POLLERR or fds[0].revents & POLLHUP or fds[0].revents & POLLNVAL) {
       int err;
-      socklen_t len= sizeof (err);
-      if (getsockopt(fds[0].fd, SOL_SOCKET, SO_ERROR, (char*)&err, &len) == -1)
-      {
-        return memcached_set_errno(*server, errno, MEMCACHED_AT, memcached_literal_param("getsockopt() errored while looking up error state from poll()"));
+      socklen_t len = sizeof(err);
+      if (getsockopt(fds[0].fd, SOL_SOCKET, SO_ERROR, (char *) &err, &len) == -1) {
+        return memcached_set_errno(
+            *server, errno, MEMCACHED_AT,
+            memcached_literal_param(
+                "getsockopt() errored while looking up error state from poll()"));
       }
 
       // We check the value to see what happened with the socket.
@@ -151,46 +108,47 @@ static memcached_return_t connect_poll(memcached_instance_st* server, const int
       {
         return MEMCACHED_SUCCESS;
       }
-      errno= err;
+      errno = err;
 
-      return memcached_set_errno(*server, err, MEMCACHED_AT, memcached_literal_param("getsockopt() found the error from poll() during connect."));
+      return memcached_set_errno(
+          *server, err, MEMCACHED_AT,
+          memcached_literal_param("getsockopt() found the error from poll() during connect."));
     }
     assert(fds[0].revents & POLLOUT);
 
-    if (fds[0].revents & POLLOUT and connection_error == EINPROGRESS)
-    {
+    if (fds[0].revents & POLLOUT and connection_error != EALREADY) {
       int err;
-      socklen_t len= sizeof(err);
-      if (getsockopt(server->fd, SOL_SOCKET, SO_ERROR, (char*)&err, &len) == -1)
-      {
+      socklen_t len = sizeof(err);
+      if (getsockopt(server->fd, SOL_SOCKET, SO_ERROR, (char *) &err, &len) == -1) {
         return memcached_set_errno(*server, errno, MEMCACHED_AT);
       }
 
-      if (err == 0)
-      {
+      if (err == 0) {
         return MEMCACHED_SUCCESS;
       }
 
-      return memcached_set_errno(*server, err, MEMCACHED_AT, memcached_literal_param("getsockopt() found the error from poll() after connect() returned EINPROGRESS."));
+      return memcached_set_errno(
+          *server, err, MEMCACHED_AT,
+          memcached_literal_param(
+              "getsockopt() found the error from poll() after connect() returned EINPROGRESS."));
     }
 
     break; // We only have the loop setup for errno types that require restart
   }
 
   // This should only be possible from ERESTART or EINTR;
-  return memcached_set_errno(*server, connection_error, MEMCACHED_AT, memcached_literal_param("connect_poll() was exhausted"));
+  return memcached_set_errno(*server, connection_error, MEMCACHED_AT,
+                             memcached_literal_param("connect_poll() was exhausted"));
 }
 
-static memcached_return_t set_hostinfo(memcached_instance_st* server)
-{
+static memcached_return_t set_hostinfo(memcached_instance_st *server) {
   assert(server->type != MEMCACHED_CONNECTION_UNIX_SOCKET);
   server->clear_addrinfo();
 
-  char str_port[MEMCACHED_NI_MAXSERV]= { 0 };
-  errno= 0;
-  int length= snprintf(str_port, MEMCACHED_NI_MAXSERV, "%u", uint32_t(server->port()));
-  if (length >= MEMCACHED_NI_MAXSERV or length <= 0 or errno != 0)
-  {
+  char str_port[MEMCACHED_NI_MAXSERV] = {0};
+  errno = 0;
+  int length = snprintf(str_port, MEMCACHED_NI_MAXSERV, "%u", uint32_t(server->port()));
+  if (length >= MEMCACHED_NI_MAXSERV or length <= 0 or errno) {
     return memcached_set_error(*server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT,
                                memcached_literal_param("snprintf(NI_MAXSERV)"));
   }
@@ -198,87 +156,78 @@ static memcached_return_t set_hostinfo(memcached_instance_st* server)
   struct addrinfo hints;
   memset(&hints, 0, sizeof(struct addrinfo));
 
-  hints.ai_family= AF_UNSPEC;
-  if (memcached_is_udp(server->root))
-  {
-    hints.ai_protocol= IPPROTO_UDP;
-    hints.ai_socktype= SOCK_DGRAM;
-  }
-  else
-  {
-    hints.ai_socktype= SOCK_STREAM;
-    hints.ai_protocol= IPPROTO_TCP;
+  hints.ai_family = AF_UNSPEC;
+  if (memcached_is_udp(server->root)) {
+    hints.ai_protocol = IPPROTO_UDP;
+    hints.ai_socktype = SOCK_DGRAM;
+  } else {
+    hints.ai_socktype = SOCK_STREAM;
+    hints.ai_protocol = IPPROTO_TCP;
   }
 
   assert(server->address_info == NULL);
   assert(server->address_info_next == NULL);
   int errcode;
   assert(server->hostname());
-  switch(errcode= getaddrinfo(server->hostname(), str_port, &hints, &server->address_info))
-  {
+  switch (errcode = getaddrinfo(server->hostname(), str_port, &hints, &server->address_info)) {
   case 0:
-    server->address_info_next= server->address_info;
-    server->state= MEMCACHED_SERVER_STATE_ADDRINFO;
+    server->address_info_next = server->address_info;
+    server->state = MEMCACHED_SERVER_STATE_ADDRINFO;
     break;
 
   case EAI_AGAIN:
-    return memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT, memcached_string_make_from_cstr(gai_strerror(errcode)));
+    return memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
+                               memcached_string_make_from_cstr(gai_strerror(errcode)));
 
   case EAI_SYSTEM:
     server->clear_addrinfo();
-    return memcached_set_errno(*server, errno, MEMCACHED_AT, memcached_literal_param("getaddrinfo(EAI_SYSTEM)"));
+    return memcached_set_errno(*server, errno, MEMCACHED_AT,
+                               memcached_literal_param("getaddrinfo(EAI_SYSTEM)"));
 
   case EAI_BADFLAGS:
     server->clear_addrinfo();
-    return memcached_set_error(*server, MEMCACHED_INVALID_ARGUMENTS, MEMCACHED_AT, memcached_literal_param("getaddrinfo(EAI_BADFLAGS)"));
+    return memcached_set_error(*server, MEMCACHED_INVALID_ARGUMENTS, MEMCACHED_AT,
+                               memcached_literal_param("getaddrinfo(EAI_BADFLAGS)"));
 
   case EAI_MEMORY:
     server->clear_addrinfo();
-    return memcached_set_error(*server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT, memcached_literal_param("getaddrinfo(EAI_MEMORY)"));
+    return memcached_set_error(*server, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT,
+                               memcached_literal_param("getaddrinfo(EAI_MEMORY)"));
 
-  default:
-    {
-      server->clear_addrinfo();
-      return memcached_set_error(*server, MEMCACHED_HOST_LOOKUP_FAILURE, MEMCACHED_AT, memcached_string_make_from_cstr(gai_strerror(errcode)));
-    }
+  default: {
+    server->clear_addrinfo();
+    return memcached_set_error(*server, MEMCACHED_HOST_LOOKUP_FAILURE, MEMCACHED_AT,
+                               memcached_string_make_from_cstr(gai_strerror(errcode)));
+  }
   }
 
   return MEMCACHED_SUCCESS;
 }
 
-static inline void set_socket_nonblocking(memcached_instance_st* server)
-{
+static inline void set_socket_nonblocking(memcached_instance_st *server) {
 #if defined(_WIN32)
-  u_long arg= 1;
-  if (ioctlsocket(server->fd, FIONBIO, &arg) == SOCKET_ERROR)
-  {
+  u_long arg = 1;
+  if (ioctlsocket(server->fd, FIONBIO, &arg) == SOCKET_ERROR) {
     memcached_set_errno(*server, get_socket_errno(), NULL);
   }
 #else
   int flags;
 
-  if (SOCK_NONBLOCK == 0)
-  {
-    do
-    {
-      flags= fcntl(server->fd, F_GETFL, 0);
+  if (SOCK_NONBLOCK == 0) {
+    do {
+      flags = fcntl(server->fd, F_GETFL, 0);
     } while (flags == -1 && (errno == EINTR || errno == EAGAIN));
 
-    if (flags == -1)
-    {
+    if (flags == -1) {
       memcached_set_errno(*server, errno, NULL);
-    }
-    else if ((flags & O_NONBLOCK) == 0)
-    {
+    } else if ((flags & O_NONBLOCK) == 0) {
       int rval;
 
-      do
-      {
-        rval= fcntl(server->fd, F_SETFL, flags | O_NONBLOCK);
+      do {
+        rval = fcntl(server->fd, F_SETFL, flags | O_NONBLOCK);
       } while (rval == -1 && (errno == EINTR or errno == EAGAIN));
 
-      if (rval == -1)
-      {
+      if (rval == -1) {
         memcached_set_errno(*server, errno, NULL);
       }
     }
@@ -286,28 +235,22 @@ static inline void set_socket_nonblocking(memcached_instance_st* server)
 #endif
 }
 
-static bool set_socket_options(memcached_instance_st* server)
-{
+static bool set_socket_options(memcached_instance_st *server) {
   assert_msg(server->fd != INVALID_SOCKET, "invalid socket was passed to set_socket_options()");
 
 #ifdef HAVE_FCNTL
   // If SOCK_CLOEXEC exists then we don't need to call the following
-  if (SOCK_CLOEXEC == 0)
-  {
-    if (FD_CLOEXEC != 0)
-    {
+  if (SOCK_CLOEXEC == 0) {
+    if (FD_CLOEXEC) {
       int flags;
-      do
-      {
-        flags= fcntl(server->fd, F_GETFD, 0);
+      do {
+        flags = fcntl(server->fd, F_GETFD, 0);
       } while (flags == -1 and (errno == EINTR or errno == EAGAIN));
 
-      if (flags != -1)
-      {
+      if (flags != -1) {
         int rval;
-        do
-        {
-          rval= fcntl (server->fd, F_SETFD, flags | FD_CLOEXEC);
+        do {
+          rval = fcntl(server->fd, F_SETFD, flags | FD_CLOEXEC);
         } while (rval == -1 && (errno == EINTR or errno == EAGAIN));
         // we currently ignore the case where rval is -1
       }
@@ -315,122 +258,108 @@ static bool set_socket_options(memcached_instance_st* server)
   }
 #endif
 
-  if (memcached_is_udp(server->root))
-  {
+  if (memcached_is_udp(server->root)) {
     return true;
   }
 
 #ifdef HAVE_SO_SNDTIMEO
-  if (server->root->snd_timeout > 0)
-  {
+  if (server->root->snd_timeout > 0) {
     struct timeval waittime;
 
-    waittime.tv_sec= server->root->snd_timeout / 1000000;
-    waittime.tv_usec= server->root->snd_timeout % 1000000;
+    waittime.tv_sec = server->root->snd_timeout / 1000000;
+    waittime.tv_usec = server->root->snd_timeout % 1000000;
 
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_SNDTIMEO,
-                          (char*)&waittime, (socklen_t)sizeof(struct timeval));
-    (void)error;
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_SNDTIMEO, (char *) &waittime,
+                           (socklen_t) sizeof(struct timeval));
+    (void) error;
     assert(error == 0);
   }
 #endif
 
 #ifdef HAVE_SO_RCVTIMEO
-  if (server->root->rcv_timeout > 0)
-  {
+  if (server->root->rcv_timeout > 0) {
     struct timeval waittime;
 
-    waittime.tv_sec= server->root->rcv_timeout / 1000000;
-    waittime.tv_usec= server->root->rcv_timeout % 1000000;
+    waittime.tv_sec = server->root->rcv_timeout / 1000000;
+    waittime.tv_usec = server->root->rcv_timeout % 1000000;
 
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_RCVTIMEO,
-                          (char*)&waittime, (socklen_t)sizeof(struct timeval));
-    (void)(error);
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_RCVTIMEO, (char *) &waittime,
+                           (socklen_t) sizeof(struct timeval));
+    (void) (error);
     assert(error == 0);
   }
 #endif
 
-
 #if defined(_WIN32)
 #else
-# if defined(SO_NOSIGPIPE)
-  if (SO_NOSIGPIPE)
-  {
-    int set= 1;
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_NOSIGPIPE, (void *)&set, sizeof(int));
+#  if defined(SO_NOSIGPIPE)
+  if (SO_NOSIGPIPE) {
+    int set = 1;
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_NOSIGPIPE, (void *) &set, sizeof(int));
 
     assert(error == 0);
 
     // This is not considered a fatal error
-    if (error == -1)
-    {
-#if 0
+    if (error == -1) {
+#    if 0
       perror("setsockopt(SO_NOSIGPIPE)");
-#endif
+#    endif
     }
   }
-# endif // SO_NOSIGPIPE
-#endif // _WIN32
+#  endif // SO_NOSIGPIPE
+#endif   // _WIN32
 
-  if (server->root->flags.no_block)
-  {
+  if (server->root->flags.no_block) {
     struct linger linger;
 
-    linger.l_onoff= 1;
-    linger.l_linger= 0; /* By default on close() just drop the socket */
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_LINGER,
-                          (char*)&linger, (socklen_t)sizeof(struct linger));
-    (void)(error);
+    linger.l_onoff = 1;
+    linger.l_linger = 0; /* By default on close() just drop the socket */
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_LINGER, (char *) &linger,
+                           (socklen_t) sizeof(struct linger));
+    (void) (error);
     assert(error == 0);
   }
 
-  if (TCP_NODELAY)
-  {
-    if (server->root->flags.tcp_nodelay)
-    {
-      int flag= 1;
+  if (TCP_NODELAY) {
+    if (server->root->flags.tcp_nodelay) {
+      int flag = 1;
 
-      int error= setsockopt(server->fd, IPPROTO_TCP, TCP_NODELAY,
-                            (char*)&flag, (socklen_t)sizeof(int));
-      (void)(error);
+      int error =
+          setsockopt(server->fd, IPPROTO_TCP, TCP_NODELAY, (char *) &flag, (socklen_t) sizeof(int));
+      (void) (error);
       assert(error == 0);
     }
   }
 
-  if (server->root->flags.tcp_keepalive)
-  {
-    int flag= 1;
+  if (server->root->flags.tcp_keepalive) {
+    int flag = 1;
 
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_KEEPALIVE,
-                          (char*)&flag, (socklen_t)sizeof(int));
-    (void)(error);
+    int error =
+        setsockopt(server->fd, SOL_SOCKET, SO_KEEPALIVE, (char *) &flag, (socklen_t) sizeof(int));
+    (void) (error);
     assert(error == 0);
   }
 
-  if (TCP_KEEPIDLE)
-  {
-    if (server->root->tcp_keepidle > 0)
-    {
-      int error= setsockopt(server->fd, IPPROTO_TCP, TCP_KEEPIDLE,
-                            (char*)&server->root->tcp_keepidle, (socklen_t)sizeof(int));
-      (void)(error);
+  if (TCP_KEEPIDLE) {
+    if (server->root->tcp_keepidle > 0) {
+      int error = setsockopt(server->fd, IPPROTO_TCP, TCP_KEEPIDLE,
+                             (char *) &server->root->tcp_keepidle, (socklen_t) sizeof(int));
+      (void) (error);
       assert(error == 0);
     }
   }
 
-  if (server->root->send_size > 0)
-  {
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_SNDBUF,
-                          (char*)&server->root->send_size, (socklen_t)sizeof(int));
-    (void)(error);
+  if (server->root->send_size > 0) {
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_SNDBUF, (char *) &server->root->send_size,
+                           (socklen_t) sizeof(int));
+    (void) (error);
     assert(error == 0);
   }
 
-  if (server->root->recv_size > 0)
-  {
-    int error= setsockopt(server->fd, SOL_SOCKET, SO_RCVBUF,
-                          (char*)&server->root->recv_size, (socklen_t)sizeof(int));
-    (void)(error);
+  if (server->root->recv_size > 0) {
+    int error = setsockopt(server->fd, SOL_SOCKET, SO_RCVBUF, (char *) &server->root->recv_size,
+                           (socklen_t) sizeof(int));
+    (void) (error);
     assert(error == 0);
   }
 
@@ -440,41 +369,36 @@ static bool set_socket_options(memcached_instance_st* server)
   return true;
 }
 
-static memcached_return_t unix_socket_connect(memcached_instance_st* server)
-{
+static memcached_return_t unix_socket_connect(memcached_instance_st *server) {
 #ifndef _WIN32
   WATCHPOINT_ASSERT(server->fd == INVALID_SOCKET);
 
   do {
-    int type= SOCK_STREAM;
-    if (SOCK_CLOEXEC != 0)
-    {
-      type|= SOCK_CLOEXEC;
+    int type = SOCK_STREAM;
+    if (SOCK_CLOEXEC) {
+      type |= SOCK_CLOEXEC;
     }
 
-    if (SOCK_NONBLOCK != 0)
-    {
-      type|= SOCK_NONBLOCK;
+    if (SOCK_NONBLOCK) {
+      type |= SOCK_NONBLOCK;
     }
 
-    if ((server->fd= socket(AF_UNIX, type, 0)) == -1)
-    {
+    if ((server->fd = socket(AF_UNIX, type, 0)) == -1) {
       return memcached_set_errno(*server, errno, NULL);
     }
 
     struct sockaddr_un servAddr;
 
-    memset(&servAddr, 0, sizeof (struct sockaddr_un));
-    servAddr.sun_family= AF_UNIX;
+    memset(&servAddr, 0, sizeof(struct sockaddr_un));
+    servAddr.sun_family = AF_UNIX;
     if (strlen(server->hostname()) >= sizeof(servAddr.sun_path)) {
-        return memcached_set_error(*server, MEMCACHED_FAIL_UNIX_SOCKET, MEMCACHED_AT);
+      return memcached_set_error(*server, MEMCACHED_FAIL_UNIX_SOCKET, MEMCACHED_AT);
     }
-    strncpy(servAddr.sun_path, server->hostname(), sizeof(servAddr.sun_path)-1); /* Copy filename */
+    strncpy(servAddr.sun_path, server->hostname(),
+            sizeof(servAddr.sun_path) - 1); /* Copy filename */
 
-    if (connect(server->fd, (struct sockaddr *)&servAddr, sizeof(servAddr)) == -1)
-    {
-      switch (errno)
-      {
+    if (connect(server->fd, (struct sockaddr *) &servAddr, sizeof(servAddr)) == -1) {
+      switch (errno) {
       case EINPROGRESS:
       case EALREADY:
       case EAGAIN:
@@ -486,11 +410,11 @@ static memcached_return_t unix_socket_connect(memcached_instance_st* server)
         continue;
 
       case EISCONN: /* We were spinning waiting on connect */
-        {
-          assert(0); // Programmer error
-          server->reset_socket();
-          continue;
-        }
+      {
+        assert(0); // Programmer error
+        server->reset_socket();
+        continue;
+      }
 
       default:
         WATCHPOINT_ERRNO(errno);
@@ -499,35 +423,33 @@ static memcached_return_t unix_socket_connect(memcached_instance_st* server)
       }
     }
   } while (0);
-  server->state= MEMCACHED_SERVER_STATE_CONNECTED;
+  server->state = MEMCACHED_SERVER_STATE_CONNECTED;
 
   WATCHPOINT_ASSERT(server->fd != INVALID_SOCKET);
 
   return MEMCACHED_SUCCESS;
 #else
-  (void)server;
+  (void) server;
   return MEMCACHED_NOT_SUPPORTED;
 #endif
 }
 
-static memcached_return_t network_connect(memcached_instance_st* server)
-{
-  bool timeout_error_occured= false;
+static memcached_return_t network_connect(memcached_instance_st *server) {
+  bool timeout_error_occured = false;
 
   WATCHPOINT_ASSERT(server->fd == INVALID_SOCKET);
   WATCHPOINT_ASSERT(server->cursor_active_ == 0);
 
   /*
-    We want to check both of these because if address_info_next has been fully tried, we want to do a new lookup to make sure we have picked up on any new DNS information.
+    We want to check both of these because if address_info_next has been fully tried, we want to do
+    a new lookup to make sure we have picked up on any new DNS information.
   */
-  if (server->address_info == NULL or server->address_info_next == NULL)
-  {
+  if (server->address_info == NULL or server->address_info_next == NULL) {
     WATCHPOINT_ASSERT(server->state == MEMCACHED_SERVER_STATE_NEW);
-    server->address_info_next= NULL;
-    memcached_return_t rc= set_hostinfo(server);
+    server->address_info_next = NULL;
+    memcached_return_t rc = set_hostinfo(server);
 
-    if (memcached_failed(rc))
-    {
+    if (memcached_failed(rc)) {
       return rc;
     }
   }
@@ -536,74 +458,67 @@ static memcached_return_t network_connect(memcached_instance_st* server)
   assert(server->address_info);
 
   /* Create the socket */
-  while (server->address_info_next and server->fd == INVALID_SOCKET)
-  {
-    int type= server->address_info_next->ai_socktype;
-    if (SOCK_CLOEXEC != 0)
-    {
-      type|= SOCK_CLOEXEC;
+  while (server->address_info_next and server->fd == INVALID_SOCKET) {
+    int type = server->address_info_next->ai_socktype;
+    if (SOCK_CLOEXEC) {
+      type |= SOCK_CLOEXEC;
     }
 
-    if (SOCK_NONBLOCK != 0)
-    {
-      type|= SOCK_NONBLOCK;
+    if (SOCK_NONBLOCK) {
+      type |= SOCK_NONBLOCK;
     }
 
-    server->fd= socket(server->address_info_next->ai_family,
-                       type,
-                       server->address_info_next->ai_protocol);
+    server->fd =
+        socket(server->address_info_next->ai_family, type, server->address_info_next->ai_protocol);
 
-    if (int(server->fd) == SOCKET_ERROR)
-    {
+    if (int(server->fd) == SOCKET_ERROR) {
       return memcached_set_errno(*server, get_socket_errno(), NULL);
     }
 
-    if (set_socket_options(server) == false)
-    {
+    if (set_socket_options(server) == false) {
       server->reset_socket();
       return MEMCACHED_CONNECTION_FAILURE;
     }
 
     /* connect to server */
-    if ((connect(server->fd, server->address_info_next->ai_addr, server->address_info_next->ai_addrlen) != SOCKET_ERROR))
+    if ((connect(server->fd, server->address_info_next->ai_addr,
+                 server->address_info_next->ai_addrlen)
+         != SOCKET_ERROR))
     {
-      server->state= MEMCACHED_SERVER_STATE_CONNECTED;
+      server->state = MEMCACHED_SERVER_STATE_CONNECTED;
       return MEMCACHED_SUCCESS;
     }
 
     /* An error occurred */
-    int local_error= get_socket_errno();
-    switch (local_error)
-    {
+    int local_error = get_socket_errno();
+    switch (local_error) {
     case ETIMEDOUT:
-      timeout_error_occured= true;
+      timeout_error_occured = true;
       break;
 
 #if EWOULDBLOCK != EAGAIN
     case EWOULDBLOCK:
 #endif
+    case EAGAIN:
     case EINPROGRESS: // nonblocking mode - first return
-    case EALREADY: // nonblocking mode - subsequent returns
-      {
-        server->events(POLLOUT);
-        server->state= MEMCACHED_SERVER_STATE_IN_PROGRESS;
-        memcached_return_t rc= connect_poll(server, local_error);
+    case EALREADY:    // nonblocking mode - subsequent returns
+    {
+      server->events(POLLOUT);
+      server->state = MEMCACHED_SERVER_STATE_IN_PROGRESS;
+      memcached_return_t rc = connect_poll(server, local_error);
 
-        if (memcached_success(rc))
-        {
-          server->state= MEMCACHED_SERVER_STATE_CONNECTED;
-          return MEMCACHED_SUCCESS;
-        }
+      if (memcached_success(rc)) {
+        server->state = MEMCACHED_SERVER_STATE_CONNECTED;
+        return MEMCACHED_SUCCESS;
+      }
 
-        // A timeout here is treated as an error, we will not retry
-        if (rc == MEMCACHED_TIMEOUT)
-        {
-          timeout_error_occured= true;
-        }
+      // A timeout here is treated as an error, we will not retry
+      if (rc == MEMCACHED_TIMEOUT) {
+        timeout_error_occured = true;
       }
-      break;
+    } break;
 
-    case EISCONN: // we are connected :-)
+    case EISCONN:           // we are connected :-)
       WATCHPOINT_ASSERT(0); // This is a programmer's error
       break;
 
@@ -622,43 +537,41 @@ static memcached_return_t network_connect(memcached_instance_st* server)
 
     WATCHPOINT_ASSERT(server->fd != INVALID_SOCKET);
     server->reset_socket();
-    server->address_info_next= server->address_info_next->ai_next;
+    server->address_info_next = server->address_info_next->ai_next;
   }
 
   WATCHPOINT_ASSERT(server->fd == INVALID_SOCKET);
 
-  if (timeout_error_occured)
-  {
+  if (timeout_error_occured) {
     server->reset_socket();
   }
 
   WATCHPOINT_STRING("Never got a good file descriptor");
 
-  if (memcached_has_current_error(*server))
-  {
+  if (memcached_has_current_error(*server)) {
     return memcached_instance_error_return(server);
   }
 
-  if (timeout_error_occured and server->state < MEMCACHED_SERVER_STATE_IN_PROGRESS)
-  {
-    return memcached_set_error(*server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
-                               memcached_literal_param("if (timeout_error_occured and server->state < MEMCACHED_SERVER_STATE_IN_PROGRESS)"));
+  if (timeout_error_occured and server->state < MEMCACHED_SERVER_STATE_IN_PROGRESS) {
+    return memcached_set_error(
+        *server, MEMCACHED_TIMEOUT, MEMCACHED_AT,
+        memcached_literal_param(
+            "if (timeout_error_occured and server->state < MEMCACHED_SERVER_STATE_IN_PROGRESS)"));
   }
 
-  return memcached_set_error(*server, MEMCACHED_CONNECTION_FAILURE, MEMCACHED_AT); /* The last error should be from connect() */
+  return memcached_set_error(*server, MEMCACHED_CONNECTION_FAILURE,
+                             MEMCACHED_AT); /* The last error should be from connect() */
 }
 
-
 /*
   backoff_handling()
 
-  Based on time/failure count fail the connect without trying. This prevents waiting in a state where
-  we get caught spending cycles just waiting.
+  Based on time/failure count fail the connect without trying. This prevents waiting in a state
+  where we get caught spending cycles just waiting.
 */
-static memcached_return_t backoff_handling(memcached_instance_st* server, bool& in_timeout)
-{
+static memcached_return_t backoff_handling(memcached_instance_st *server, bool &in_timeout) {
   struct timeval curr_time;
-  bool _gettime_success= (gettimeofday(&curr_time, NULL) == 0);
+  bool _gettime_success = (gettimeofday(&curr_time, NULL) == 0);
 
   /*
     If we hit server_failure_limit then something is completely wrong about the server.
@@ -666,106 +579,95 @@ static memcached_return_t backoff_handling(memcached_instance_st* server, bool&
     1) If autoeject is enabled we do that.
     2) If not? We go into timeout again, there is much else to do :(
   */
-  if (server->server_failure_counter >= server->root->server_failure_limit)
-  {
+  if (server->server_failure_counter >= server->root->server_failure_limit) {
     /*
       We just auto_eject if we hit this point
     */
-    if (_is_auto_eject_host(server->root))
-    {
+    if (_is_auto_eject_host(server->root)) {
       set_last_disconnected_host(server);
 
       // Retry dead servers if requested
-      if (_gettime_success and server->root->dead_timeout > 0)
-      {
-        server->next_retry= curr_time.tv_sec +server->root->dead_timeout;
+      if (_gettime_success and server->root->dead_timeout > 0) {
+        server->next_retry = curr_time.tv_sec + server->root->dead_timeout;
 
         // We only retry dead servers once before assuming failure again
-        server->server_failure_counter= server->root->server_failure_limit -1;
+        server->server_failure_counter = server->root->server_failure_limit - 1;
       }
 
       memcached_return_t rc;
-      if (memcached_failed(rc= run_distribution((memcached_st *)server->root)))
-      {
-        return memcached_set_error(*server, rc, MEMCACHED_AT, memcached_literal_param("Backoff handling failed during run_distribution"));
+      if (memcached_failed(rc = run_distribution((memcached_st *) server->root))) {
+        return memcached_set_error(
+            *server, rc, MEMCACHED_AT,
+            memcached_literal_param("Backoff handling failed during run_distribution"));
       }
 
       return memcached_set_error(*server, MEMCACHED_SERVER_MARKED_DEAD, MEMCACHED_AT);
     }
 
-    server->state= MEMCACHED_SERVER_STATE_IN_TIMEOUT;
+    server->state = MEMCACHED_SERVER_STATE_IN_TIMEOUT;
 
     // Sanity check/setting
-    if (server->next_retry == 0)
-    {
-      server->next_retry= 1;
+    if (server->next_retry == 0) {
+      server->next_retry = 1;
     }
   }
 
-  if (server->state == MEMCACHED_SERVER_STATE_IN_TIMEOUT)
-  {
+  if (server->state == MEMCACHED_SERVER_STATE_IN_TIMEOUT) {
     /*
       If next_retry is less then our current time, then we reset and try everything again.
     */
-    if (_gettime_success and server->next_retry < curr_time.tv_sec)
-    {
-      server->state= MEMCACHED_SERVER_STATE_NEW;
-      server->server_timeout_counter= 0;
-    }
-    else
-    {
+    if (_gettime_success and server->next_retry < curr_time.tv_sec) {
+      server->state = MEMCACHED_SERVER_STATE_NEW;
+      server->server_timeout_counter = 0;
+    } else {
       return memcached_set_error(*server, MEMCACHED_SERVER_TEMPORARILY_DISABLED, MEMCACHED_AT);
     }
 
-    in_timeout= true;
+    in_timeout = true;
   }
 
   return MEMCACHED_SUCCESS;
 }
 
-static memcached_return_t _memcached_connect(memcached_instance_st* server, const bool set_last_disconnected)
-{
+static memcached_return_t _memcached_connect(memcached_instance_st *server,
+                                             const bool set_last_disconnected) {
   assert(server);
-  if (server->fd != INVALID_SOCKET)
-  {
+  if (server->fd != INVALID_SOCKET) {
     return MEMCACHED_SUCCESS;
   }
 
   LIBMEMCACHED_MEMCACHED_CONNECT_START();
 
-  bool in_timeout= false;
+  bool in_timeout = false;
   memcached_return_t rc;
-  if (memcached_failed(rc= backoff_handling(server, in_timeout)))
-  {
+  if (memcached_failed(rc = backoff_handling(server, in_timeout))) {
     set_last_disconnected_host(server);
     return rc;
   }
 
-  if (LIBMEMCACHED_WITH_SASL_SUPPORT and server->root->sasl.callbacks and memcached_is_udp(server->root))
+  if (LIBMEMCACHED_WITH_SASL_SUPPORT and server->root->sasl.callbacks
+      and memcached_is_udp(server->root))
   {
-    return memcached_set_error(*server, MEMCACHED_INVALID_HOST_PROTOCOL, MEMCACHED_AT, memcached_literal_param("SASL is not supported for UDP connections"));
+    return memcached_set_error(
+        *server, MEMCACHED_INVALID_HOST_PROTOCOL, MEMCACHED_AT,
+        memcached_literal_param("SASL is not supported for UDP connections"));
   }
 
-  if (server->hostname()[0] == '/')
-  {
-    server->type= MEMCACHED_CONNECTION_UNIX_SOCKET;
+  if (server->hostname()[0] == '/') {
+    server->type = MEMCACHED_CONNECTION_UNIX_SOCKET;
   }
 
   /* We need to clean up the multi startup piece */
-  switch (server->type)
-  {
+  switch (server->type) {
   case MEMCACHED_CONNECTION_UDP:
   case MEMCACHED_CONNECTION_TCP:
-    rc= network_connect(server);
+    rc = network_connect(server);
 
 #if defined(LIBMEMCACHED_WITH_SASL_SUPPORT)
-    if (LIBMEMCACHED_WITH_SASL_SUPPORT)
-    {
-      if (server->fd != INVALID_SOCKET and server->root->sasl.callbacks)
-      {
-        rc= memcached_sasl_authenticate_connection(server);
-        if (memcached_failed(rc) and server->fd != INVALID_SOCKET)
-        {
+    if (LIBMEMCACHED_WITH_SASL_SUPPORT) {
+      if (server->fd != INVALID_SOCKET and server->root->sasl.callbacks) {
+        rc = memcached_sasl_authenticate_connection(server);
+        if (memcached_failed(rc) and server->fd != INVALID_SOCKET) {
           WATCHPOINT_ASSERT(server->fd != INVALID_SOCKET);
           server->reset_socket();
         }
@@ -775,44 +677,38 @@ static memcached_return_t _memcached_connect(memcached_instance_st* server, cons
     break;
 
   case MEMCACHED_CONNECTION_UNIX_SOCKET:
-    rc= unix_socket_connect(server);
+    rc = unix_socket_connect(server);
     break;
   }
 
-  if (memcached_success(rc))
-  {
+  if (memcached_success(rc)) {
     server->mark_server_as_clean();
     memcached_version_instance(server);
     return rc;
-  }
-  else if (set_last_disconnected)
-  {
+  } else if (set_last_disconnected) {
     set_last_disconnected_host(server);
-    if (memcached_has_current_error(*server))
-    {
+    if (memcached_has_current_error(*server)) {
       memcached_mark_server_for_timeout(server);
       assert(memcached_failed(memcached_instance_error_return(server)));
-    }
-    else
-    {
+    } else {
       memcached_set_error(*server, rc, MEMCACHED_AT);
       memcached_mark_server_for_timeout(server);
     }
 
     LIBMEMCACHED_MEMCACHED_CONNECT_END();
 
-    if (in_timeout)
-    {
+    if (in_timeout) {
       char buffer[1024];
-      int snprintf_length= snprintf(buffer, sizeof(buffer), "%s:%d", server->hostname(), int(server->port()));
-      return memcached_set_error(*server, MEMCACHED_SERVER_TEMPORARILY_DISABLED, MEMCACHED_AT, buffer, snprintf_length);
+      int snprintf_length =
+          snprintf(buffer, sizeof(buffer), "%s:%d", server->hostname(), int(server->port()));
+      return memcached_set_error(*server, MEMCACHED_SERVER_TEMPORARILY_DISABLED, MEMCACHED_AT,
+                                 buffer, snprintf_length);
     }
   }
 
   return rc;
 }
 
-memcached_return_t memcached_connect(memcached_instance_st* server)
-{
+memcached_return_t memcached_connect(memcached_instance_st *server) {
   return _memcached_connect(server, true);
 }