HSET returns error when value is modified and shouldn't

[GoogleCodeExporter]

When trying to modify a previously set value with hset the condition


  if (rc == 0 && rhnd->reply.integer == 0)
    rc = -1;

  return rc;

Will return error. 

This is a wrong behaviour in credis lib as long as i know. 

Redis return values for hset specifies that

    rhnd->reply.integer value is 1 for new set values, and 0 for modified values.

This patch avoid that hset return error when a value is modified by hset 
function.

I have noticed same behaviour is present in zadd. Same patch can be applied to 
that function too. 

Maybe other functions are affected too.

Original issue reported on code.google.com by [email protected] on 8 May 2012 at 7:18

Attachments:

• credis.c.patch