#include <libmemcached/common.h>
static memcached_return_t text_incr_decr(memcached_st *ptr,
- const char *verb,
+ const bool is_incr,
const char *group_key, size_t group_key_length,
const char *key, size_t key_length,
uint64_t offset,
char buffer[MEMCACHED_DEFAULT_COMMAND_SIZE];
uint32_t server_key;
memcached_server_write_instance_st instance;
- bool no_reply= ptr->flags.no_reply;
+
+ // Invert the logic to make it simpler to read the code
+ bool reply= (ptr->flags.no_reply) ? false : true;
if (memcached_failed(memcached_key_test(*ptr, (const char **)&key, &key_length, 1)))
{
server_key= memcached_generate_hash_with_redistribution(ptr, group_key, group_key_length);
instance= memcached_server_instance_fetch(ptr, server_key);
- int send_length;
- send_length= snprintf(buffer, MEMCACHED_DEFAULT_COMMAND_SIZE,
- "%s %.*s%.*s %" PRIu64 "%s\r\n", verb,
- memcached_print_array(ptr->prefix_key),
- (int)key_length, key,
- offset, no_reply ? " noreply" : "");
+ int send_length= snprintf(buffer, sizeof(buffer), " %" PRIu64, offset);
if (send_length >= MEMCACHED_DEFAULT_COMMAND_SIZE || send_length < 0)
{
return memcached_set_error(*ptr, MEMCACHED_MEMORY_ALLOCATION_FAILURE, MEMCACHED_AT,
memcached_literal_param("snprintf(MEMCACHED_DEFAULT_COMMAND_SIZE)"));
}
- memcached_return_t rc= memcached_do(instance, buffer, (size_t)send_length, true);
- if (no_reply or memcached_failed(rc))
+ struct libmemcached_io_vector_st vector[]=
+ {
+ { memcached_literal_param("incr ") },
+ { memcached_array_string(ptr->_namespace), memcached_array_size(ptr->_namespace) },
+ { key, key_length },
+ { buffer, send_length },
+ { " noreply", reply ? 0 : memcached_literal_param_size(" noreply") },
+ { memcached_literal_param("\r\n") }
+ };
+
+ if (is_incr == false)
+ {
+ vector[0].buffer= "decr ";
+ }
+
+ memcached_return_t rc= memcached_vdo(instance, vector, 6, true);
+ if (reply == false or memcached_failed(rc))
+ {
return rc;
+ }
- rc= memcached_response(instance, buffer, MEMCACHED_DEFAULT_COMMAND_SIZE, NULL);
+ rc= memcached_response(instance, buffer, sizeof(buffer), NULL);
+
+ if (rc != MEMCACHED_SUCCESS)
+ {
+ return memcached_set_error(*instance, rc, MEMCACHED_AT);
+ }
/*
So why recheck responce? Because the protocol is brain dead :)
{
bool no_reply= ptr->flags.no_reply;
- if (memcached_server_count(ptr) == 0)
- return memcached_set_error(*ptr, MEMCACHED_NO_SERVERS, MEMCACHED_AT);
-
uint32_t server_key= memcached_generate_hash_with_redistribution(ptr, group_key, group_key_length);
memcached_server_write_instance_st instance= memcached_server_instance_fetch(ptr, server_key);
request.message.header.request.magic= PROTOCOL_BINARY_REQ;
request.message.header.request.opcode= cmd;
- request.message.header.request.keylen= htons((uint16_t)(key_length + memcached_array_size(ptr->prefix_key)));
+ request.message.header.request.keylen= htons((uint16_t)(key_length + memcached_array_size(ptr->_namespace)));
request.message.header.request.extlen= 20;
request.message.header.request.datatype= PROTOCOL_BINARY_RAW_BYTES;
- request.message.header.request.bodylen= htonl((uint32_t)(key_length + memcached_array_size(ptr->prefix_key) +request.message.header.request.extlen));
+ request.message.header.request.bodylen= htonl((uint32_t)(key_length + memcached_array_size(ptr->_namespace) +request.message.header.request.extlen));
request.message.body.delta= memcached_htonll(offset);
request.message.body.initial= memcached_htonll(initial);
request.message.body.expiration= htonl((uint32_t) expiration);
struct libmemcached_io_vector_st vector[]=
{
- { sizeof(request.bytes), request.bytes },
- { memcached_array_size(ptr->prefix_key), memcached_array_string(ptr->prefix_key) },
- { key_length, key }
+ { request.bytes, sizeof(request.bytes) },
+ { memcached_array_string(ptr->_namespace), memcached_array_size(ptr->_namespace) },
+ { key, key_length }
};
memcached_return_t rc;
}
if (no_reply)
+ {
return MEMCACHED_SUCCESS;
+ }
return memcached_response(instance, (char*)value, sizeof(*value), NULL);
}
{
memcached_return_t rc;
uint64_t local_value;
- if (not value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
if (memcached_failed(rc= initialize_query(ptr)))
{
}
else
{
- rc= text_incr_decr(ptr, "incr", group_key, group_key_length, key, key_length, offset, value);
+ rc= text_incr_decr(ptr, true, group_key, group_key_length, key, key_length, offset, value);
}
LIBMEMCACHED_MEMCACHED_INCREMENT_END();
uint64_t *value)
{
uint64_t local_value;
- if (not value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
memcached_return_t rc;
if (memcached_failed(rc= initialize_query(ptr)))
}
else
{
- rc= text_incr_decr(ptr, "decr", group_key, group_key_length, key, key_length, offset, value);
+ rc= text_incr_decr(ptr, false, group_key, group_key_length, key, key_length, offset, value);
}
LIBMEMCACHED_MEMCACHED_DECREMENT_END();
uint64_t *value)
{
uint64_t local_value;
- if (! value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
return memcached_increment_with_initial_by_key(ptr, key, key_length,
key, key_length,
uint64_t *value)
{
uint64_t local_value;
- if (not value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
memcached_return_t rc;
if (memcached_failed(rc= initialize_query(ptr)))
LIBMEMCACHED_MEMCACHED_INCREMENT_WITH_INITIAL_START();
if (ptr->flags.binary_protocol)
+ {
rc= binary_incr_decr(ptr, PROTOCOL_BINARY_CMD_INCREMENT,
group_key, group_key_length, key, key_length,
offset, initial, (uint32_t)expiration,
value);
+ }
else
+ {
rc= MEMCACHED_PROTOCOL_ERROR;
+ }
LIBMEMCACHED_MEMCACHED_INCREMENT_WITH_INITIAL_END();
uint64_t *value)
{
uint64_t local_value;
- if (! value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
return memcached_decrement_with_initial_by_key(ptr, key, key_length,
key, key_length,
uint64_t *value)
{
uint64_t local_value;
- if (not value)
+ if (value == NULL)
+ {
value= &local_value;
+ }
memcached_return_t rc;
if (memcached_failed(rc= memcached_validate_key_length(key_length, ptr->flags.binary_protocol)))
projects / awesomized / libmemcached / blobdiff