Speed up host lookup for cases where we don't need to do it, or where it ends up...

[awesomized/libmemcached] / libmemcachedutil / ping.cc

diff --git a/libmemcachedutil/ping.cc b/libmemcachedutil/ping.cc
index 0154b530d5ba8295345974ab05cc3feb7800fbe2..e2831beb89f1bd134c06120aad49f26e9166da8b 100644 (file)
--- a/libmemcachedutil/ping.cc
+++ b/libmemcachedutil/ping.cc
@@ -54,30 +54,31 @@ bool libmemcached_util_ping(const char *hostname, in_port_t port, memcached_retu
     return false;
   }
 
-  (void)memcached_behavior_set(memc_ptr, MEMCACHED_BEHAVIOR_CONNECT_TIMEOUT, 400000);
-
-  memcached_return_t rc= memcached_server_add(memc_ptr, hostname, port);
-  if (memcached_success(rc))
-  {
-    rc= memcached_version(memc_ptr);
-  }
-
-  if (memcached_failed(rc) and rc == MEMCACHED_SOME_ERRORS)
+  if (memcached_success((*ret= memcached_behavior_set(memc_ptr, MEMCACHED_BEHAVIOR_CONNECT_TIMEOUT, 400000))))
   {
-    memcached_server_instance_st instance=
-      memcached_server_instance_by_position(memc_ptr, 0);
+    memcached_return_t rc= memcached_server_add(memc_ptr, hostname, port);
+    if (memcached_success(rc))
+    {
+      rc= memcached_version(memc_ptr);
+    }
 
-    assert_msg(instance and memcached_server_error(instance), " ");
-    if (instance and memcached_server_error(instance))
+    if (memcached_failed(rc) and rc == MEMCACHED_SOME_ERRORS)
     {
-      rc= memcached_server_error_return(instance);
+      memcached_server_instance_st instance=
+        memcached_server_instance_by_position(memc_ptr, 0);
+
+      assert_msg(instance and memcached_server_error(instance), " ");
+      if (instance and memcached_server_error(instance))
+      {
+        rc= memcached_server_error_return(instance);
+      }
     }
+
+    *ret= rc;
   }
   memcached_free(memc_ptr);
 
-  *ret= rc;
-
-  return memcached_success(rc);
+  return memcached_success(*ret);
 }
 
 bool libmemcached_util_ping2(const char *hostname, in_port_t port, const char *username, const char *password,  memcached_return_t *ret)