Switch verbose to vector.
authorBrian Aker <brian@tangent.org>
Thu, 22 Dec 2011 17:02:05 +0000 (09:02 -0800)
committerBrian Aker <brian@tangent.org>
Thu, 22 Dec 2011 17:02:05 +0000 (09:02 -0800)
libmemcached/response.cc
libmemcached/verbosity.cc

index 775102fa63f56e5d38b9c919e68f8e874254f5d1..dbc4e5d4b66d66432438da6407bbf1e3d7f0b360 100644 (file)
@@ -354,7 +354,7 @@ static memcached_return_t textual_read_one_response(memcached_server_write_insta
       {
         return MEMCACHED_END;
       }
-      else if (buffer[1] == 'R')
+      else if (buffer[1] == 'R' and buffer[2] == 'R' and buffer[3] == 'O' and buffer[4] == 'R')
       {
         return MEMCACHED_PROTOCOL_ERROR;
       }
@@ -380,7 +380,7 @@ static memcached_return_t textual_read_one_response(memcached_server_write_insta
     }
     return MEMCACHED_UNKNOWN_READ_FAILURE;
 
-  case 'I': /* CLIENT ERROR */
+  case 'I': /* ITEM */
     /* We add back in one because we will need to search for END */
     memcached_server_response_increment(ptr);
     return MEMCACHED_ITEM;
index 5564c7a90f7b894cb88b8349749daf9ddcbd40fa..e2aac3c74ba88c24bee5560781c2d8cb66bfa616 100644 (file)
@@ -47,7 +47,7 @@ static memcached_return_t _set_verbosity(const memcached_st *,
                                          const memcached_server_st *server,
                                          void *context)
 {
 struct context_st *execute= (struct context_st *)context;
const libmemcached_io_vector_st *execute= (const libmemcached_io_vector_st *)context;
 
   memcached_st local_memc;
   memcached_st *memc_ptr= memcached_create(&local_memc);
@@ -56,16 +56,15 @@ static memcached_return_t _set_verbosity(const memcached_st *,
 
   if (rc == MEMCACHED_SUCCESS)
   {
-    char buffer[MEMCACHED_DEFAULT_COMMAND_SIZE];
-
     memcached_server_write_instance_st instance= memcached_server_instance_fetch(memc_ptr, 0);
 
 
-    rc= memcached_do(instance, execute->buffer, execute->length, true);
+    rc= memcached_vdo(instance, execute, 3, true);
 
     if (rc == MEMCACHED_SUCCESS)
     {
-      rc= memcached_response(instance, buffer, MEMCACHED_DEFAULT_COMMAND_SIZE, NULL);
+      char buffer[MEMCACHED_DEFAULT_COMMAND_SIZE];
+      rc= memcached_response(instance, buffer, sizeof(buffer), NULL);
     }
   }
 
@@ -76,22 +75,25 @@ static memcached_return_t _set_verbosity(const memcached_st *,
 
 memcached_return_t memcached_verbosity(memcached_st *ptr, uint32_t verbosity)
 {
-  int send_length;
   memcached_server_fn callbacks[1];
 
   char buffer[MEMCACHED_DEFAULT_COMMAND_SIZE];
 
-  send_length= snprintf(buffer, MEMCACHED_DEFAULT_COMMAND_SIZE,
-                                 "verbosity %u\r\n", verbosity);
-  if (send_length >= MEMCACHED_DEFAULT_COMMAND_SIZE || send_length < 0)
+  int send_length= snprintf(buffer, sizeof(buffer), "%u", verbosity);
+  if (send_length >= MEMCACHED_DEFAULT_COMMAND_SIZE or send_length < 0)
   {
     return memcached_set_error(*ptr, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT, 
                                memcached_literal_param("snprintf(MEMCACHED_DEFAULT_COMMAND_SIZE)"));
   }
 
-  struct context_st context = { (size_t)send_length, buffer };
+  struct libmemcached_io_vector_st vector[]=
+  {
+    { memcached_literal_param("verbosity ") },
+    { buffer, send_length },
+    { memcached_literal_param("\r\n") }
+  };
 
   callbacks[0]= _set_verbosity;
 
-  return memcached_server_cursor(ptr, callbacks, &context, 1);
+  return memcached_server_cursor(ptr, callbacks, vector, 1);
 }