X-Git-Url: https://git.m6w6.name/?a=blobdiff_plain;f=libmemcached%2Fmemcached_storage.c;h=53d79822ae633f5b7eabefd1eea55fc77bac4025;hb=e115444989a124cc1a433ea20c13e877317bb0ad;hp=fe8024d305e0e72bf9b3b9f0c334e2ebe2b8a890;hpb=7afbf5c09228f6614b8831ec5f374971a38bbf9f;p=awesomized%2Flibmemcached diff --git a/libmemcached/memcached_storage.c b/libmemcached/memcached_storage.c index fe8024d3..53d79822 100644 --- a/libmemcached/memcached_storage.c +++ b/libmemcached/memcached_storage.c @@ -37,7 +37,7 @@ static char *storage_op_string(memcached_storage_action verb) return "cas "; default: return "tosserror"; /* This is impossible, fixes issue for compiler warning in VisualStudio */ - }; + } /* NOTREACHED */ } @@ -384,6 +384,9 @@ static inline uint8_t get_com_code(memcached_storage_action verb, bool noreply) case PREPEND_OP: ret=PROTOCOL_BINARY_CMD_PREPENDQ; break; + default: + WATCHPOINT_ASSERT(verb); + break; } else switch (verb) @@ -404,6 +407,9 @@ static inline uint8_t get_com_code(memcached_storage_action verb, bool noreply) case PREPEND_OP: ret=PROTOCOL_BINARY_CMD_PREPEND; break; + default: + WATCHPOINT_ASSERT(verb); + break; } return ret; @@ -470,11 +476,11 @@ static memcached_return memcached_send_binary(memcached_st *ptr, return MEMCACHED_WRITE_FAILURE; } - if (verb == SET_OP && ptr->number_of_replicas > 0) + unlikely (verb == SET_OP && ptr->number_of_replicas > 0) { request.message.header.request.opcode= PROTOCOL_BINARY_CMD_SETQ; - for (int x= 0; x < ptr->number_of_replicas; ++x) + for (uint32_t x= 0; x < ptr->number_of_replicas; x++) { ++server_key; if (server_key == ptr->number_of_hosts) @@ -485,7 +491,9 @@ static memcached_return memcached_send_binary(memcached_st *ptr, send_length, 0) != MEMCACHED_SUCCESS) || (memcached_io_write(srv, key, key_length, 0) == -1) || (memcached_io_write(srv, value, value_length, flush) == -1)) - memcached_io_reset(server); + memcached_io_reset(srv); + else + memcached_server_response_decrement(srv); } }