fluencelabs/redis
1
0
Fork 0
mirror of https://github.com/fluencelabs/redis synced 2025-03-19 17:10:50 +00:00
Code Issues Projects Releases Wiki Activity

Geo: fix GEOHASH return value for consistency.

Browse Source 
The same thing observed in #3551 by gnethercutt also fixed for
GEOHASH as the original PR did.
This commit is contained in:
antirez 2016-12-20 10:19:06 +01:00
parent 913070a9e8
commit 0f72257049
1 changed files with 3 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

7
src/geo.c
View File

@ -684,16 +684,15 @@ void geohashCommand(client *c) {
int j; int j;
/* Look up the requested zset */ /* Look up the requested zset */
robj *zobj = NULL; robj *zobj = lookupKeyRead(c->db, c->argv[1]);
if ((zobj = lookupKeyReadOrReply(c, c->argv[1], shared.emptymultibulk)) if (zobj && checkType(c, zobj, OBJ_ZSET)) return;
== NULL || checkType(c, zobj, OBJ_ZSET)) return;
/* Geohash elements one after the other, using a null bulk reply for /* Geohash elements one after the other, using a null bulk reply for
* missing elements. */ * missing elements. */
addReplyMultiBulkLen(c,c->argc-2); addReplyMultiBulkLen(c,c->argc-2);
for (j = 2; j < c->argc; j++) { for (j = 2; j < c->argc; j++) {
double score; double score;
if (zsetScore(zobj, c->argv[j]->ptr, &score) == C_ERR) { if (!zobj || zsetScore(zobj, c->argv[j]->ptr, &score) == C_ERR) {
addReply(c,shared.nullbulk); addReply(c,shared.nullbulk);
} else { } else {
/* The internal format we use for geocoding is a bit different /* The internal format we use for geocoding is a bit different