From 2002218981c4aa6ab8a55e9513962adc2268b657 Mon Sep 17 00:00:00 2001 From: Pieter Noordhuis Date: Thu, 21 Apr 2011 14:31:02 +0200 Subject: [PATCH] Remove useless casts --- hiredis.c | 17 +++++------------ hiredis.h | 10 +++++----- 2 files changed, 10 insertions(+), 17 deletions(-) diff --git a/hiredis.c b/hiredis.c index d93b38f..428678b 100644 --- a/hiredis.c +++ b/hiredis.c @@ -544,8 +544,7 @@ redisReader *redisReplyReaderCreate(void) { /* Set the function set to build the reply. Returns REDIS_OK when there * is no temporary object and it can be set, REDIS_ERR otherwise. */ -int redisReplyReaderSetReplyObjectFunctions(redisReader *reader, redisReplyObjectFunctions *fn) { - redisReader *r = reader; +int redisReplyReaderSetReplyObjectFunctions(redisReader *r, redisReplyObjectFunctions *fn) { if (r->reply == NULL) { r->fn = fn; return REDIS_OK; @@ -555,8 +554,7 @@ int redisReplyReaderSetReplyObjectFunctions(redisReader *reader, redisReplyObjec /* Set the private data field that is used in the read tasks. This argument can * be used to curry arbitrary data to the custom reply object functions. */ -int redisReplyReaderSetPrivdata(redisReader *reader, void *privdata) { - redisReader *r = reader; +int redisReplyReaderSetPrivdata(redisReader *r, void *privdata) { if (r->reply == NULL) { r->privdata = privdata; return REDIS_OK; @@ -564,8 +562,7 @@ int redisReplyReaderSetPrivdata(redisReader *reader, void *privdata) { return REDIS_ERR; } -void redisReplyReaderFree(redisReader *reader) { - redisReader *r = reader; +void redisReplyReaderFree(redisReader *r) { if (r->reply != NULL && r->fn && r->fn->freeObject) r->fn->freeObject(r->reply); if (r->buf != NULL) @@ -573,9 +570,7 @@ void redisReplyReaderFree(redisReader *reader) { free(r); } -void redisReplyReaderFeed(redisReader *reader, const char *buf, size_t len) { - redisReader *r = reader; - +void redisReplyReaderFeed(redisReader *r, const char *buf, size_t len) { /* Copy the provided buffer. */ if (buf != NULL && len >= 1) { /* Destroy internal buffer when it is empty and is quite large. */ @@ -590,9 +585,7 @@ void redisReplyReaderFeed(redisReader *reader, const char *buf, size_t len) { } } -int redisReplyReaderGetReply(redisReader *reader, void **reply) { - redisReader *r = reader; - +int redisReplyReaderGetReply(redisReader *r, void **reply) { /* Default target pointer to NULL. */ if (reply != NULL) *reply = NULL; diff --git a/hiredis.h b/hiredis.h index 55311ba..6a2257f 100644 --- a/hiredis.h +++ b/hiredis.h @@ -133,11 +133,11 @@ typedef struct redisReader { void freeReplyObject(void *reply); redisReader *redisReplyReaderCreate(void); -int redisReplyReaderSetReplyObjectFunctions(redisReader *reader, redisReplyObjectFunctions *fn); -int redisReplyReaderSetPrivdata(redisReader *reader, void *privdata); -void redisReplyReaderFree(redisReader *ptr); -void redisReplyReaderFeed(redisReader *reader, const char *buf, size_t len); -int redisReplyReaderGetReply(redisReader *reader, void **reply); +int redisReplyReaderSetReplyObjectFunctions(redisReader *r, redisReplyObjectFunctions *fn); +int redisReplyReaderSetPrivdata(redisReader *r, void *privdata); +void redisReplyReaderFree(redisReader *r); +void redisReplyReaderFeed(redisReader *r, const char *buf, size_t len); +int redisReplyReaderGetReply(redisReader *r, void **reply); /* Backwards compatibility, can be removed on big version bump. */ #define redisReplyReaderGetObject(_r) (((redisReader*)(r))->reply)