move statement in if condition for zremrangeGenericCommand function (#2060)

hwware · web-flow · commit 2205bf99cb86 · 2025-05-09T13:41:58.000-04:00
It is similar to PR #2001, this PR goal is to make logic clearer Signed-off-by: hwware <wen.hui.ware@gmail.com>
diff --git a/src/t_zset.c b/src/t_zset.c
@@ -2011,8 +2011,8 @@ void zremrangeGenericCommand(client *c, zrange_type rangetype) {
         signalModifiedKey(c, c->db, key);
         notifyKeyspaceEvent(NOTIFY_ZSET, notify_type, key, c->db->id);
         if (keyremoved) notifyKeyspaceEvent(NOTIFY_GENERIC, "del", key, c->db->id);
+        server.dirty += deleted;
     }
-    server.dirty += deleted;
     addReplyLongLong(c, deleted);
 
 cleanup:

Original file line number	Diff line number	Diff line change
`@@ -2011,8 +2011,8 @@ void zremrangeGenericCommand(client *c, zrange_type rangetype) {`
`2011`	`2011`	`signalModifiedKey(c, c->db, key);`
`2012`	`2012`	`notifyKeyspaceEvent(NOTIFY_ZSET, notify_type, key, c->db->id);`
`2013`	`2013`	`if (keyremoved) notifyKeyspaceEvent(NOTIFY_GENERIC, "del", key, c->db->id);`
	`2014`	`+ server.dirty += deleted;`
`2014`	`2015`	`}`
`2015`		`- server.dirty += deleted;`
`2016`	`2016`	`addReplyLongLong(c, deleted);`
`2017`	`2017`
`2018`	`2018`	`cleanup:`