Fix for memcapable (it should not be looking at the local server automatically)
authorBrian Aker <brian@tangent.org>
Fri, 3 May 2013 08:44:02 +0000 (04:44 -0400)
committerBrian Aker <brian@tangent.org>
Fri, 3 May 2013 08:44:02 +0000 (04:44 -0400)
clients/memcapable.cc
libmemcached-1.0/defaults.h

index f914520658aa0324a5ba9b63c270aa231f0b8630..3e9bfabfbf2e9b63136353376b7e7b9359cef4d8 100644 (file)
@@ -186,8 +186,9 @@ static memcached_socket_t connect_server(const char *hostname, const char *port)
       }
     }
     else
-      fprintf(stderr, "Failed to create socket: %s\n",
-              strerror(get_socket_errno()));
+    {
+      fprintf(stderr, "Failed to create socket: %s\n", strerror(get_socket_errno()));
+    }
 
     freeaddrinfo(ai);
   }
@@ -2078,8 +2079,8 @@ int main(int argc, char **argv)
   struct test_type_st tests= { true, true };
   int total= 0;
   int failed= 0;
-  const char *hostname= "localhost";
-  const char *port= "11211";
+  const char *hostname= NULL;
+  const char *port= MEMCACHED_DEFAULT_PORT_STRING;
   int cmd;
   bool prompt= false;
   const char *testname= NULL;
@@ -2147,6 +2148,12 @@ int main(int argc, char **argv)
     }
   }
 
+  if (hostname)
+  {
+    fprintf(stderr, "No hostname was provided.\n");
+    return EXIT_FAILURE;
+  }
+
   initialize_sockets();
   sock= connect_server(hostname, port);
   if (sock == INVALID_SOCKET)
index 38c29ba38112a47aab62714e1271b0395e948825..177e18cf6b61ca91121e5ece1a2f5d37fe6ce97d 100644 (file)
@@ -40,6 +40,7 @@
 
 /* Public defines */
 #define MEMCACHED_DEFAULT_PORT 11211
+#define MEMCACHED_DEFAULT_PORT_STRING "11211"
 #define MEMCACHED_POINTS_PER_SERVER 100
 #define MEMCACHED_POINTS_PER_SERVER_KETAMA 160
 #define MEMCACHED_CONTINUUM_SIZE MEMCACHED_POINTS_PER_SERVER*100 /* This would then set max hosts to 100 */