From aacecbc997b9678ba59eb9ef487e5cf19078a055 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=E8=B5=B5=E7=A3=8A?= Date: Sun, 11 Feb 2018 21:02:07 +0800 Subject: [PATCH] Remove updateLFU() in dbOverwrite(). --- src/db.c | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/src/db.c b/src/db.c index 203b2984..5d540d21 100644 --- a/src/db.c +++ b/src/db.c @@ -187,9 +187,6 @@ void dbOverwrite(redisDb *db, robj *key, robj *val) { int saved_lru = old->lru; dictReplace(db->dict, key->ptr, val); val->lru = saved_lru; - /* LFU should be not only copied but also updated - * when a key is overwritten. */ - updateLFU(val); } else { dictReplace(db->dict, key->ptr, val); } @@ -1340,7 +1337,7 @@ int *georadiusGetKeys(struct redisCommand *cmd, robj **argv, int argc, int *numk for (i = 5; i < argc; i++) { char *arg = argv[i]->ptr; /* For the case when user specifies both "store" and "storedist" options, the - * second key specified would override the first key. This behavior is kept + * second key specified would override the first key. This behavior is kept * the same as in georadiusCommand method. */ if ((!strcasecmp(arg, "store") || !strcasecmp(arg, "storedist")) && ((i+1) < argc)) { @@ -1361,7 +1358,7 @@ int *georadiusGetKeys(struct redisCommand *cmd, robj **argv, int argc, int *numk if(num > 1) { keys[1] = stored_key; } - *numkeys = num; + *numkeys = num; return keys; }