summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorDvir Volk <dvirsky@gmail.com>2016-05-19 12:16:14 +0300
committerDvir Volk <dvirsky@gmail.com>2016-05-19 12:16:14 +0300
commit46b07cbb5c52a6a9321ab8c2134d3c6be9ddae86 (patch)
tree2b88e65472695c1d4508de7c34858ee31bb5bd6e
parentb09a6b6a5d7c7c2de81325b3c1ab687536059009 (diff)
downloadredis-46b07cbb5c52a6a9321ab8c2134d3c6be9ddae86.tar.gz
Optimized autoMemoryFreed loop
-rw-r--r--src/module.c5
1 files changed, 4 insertions, 1 deletions
diff --git a/src/module.c b/src/module.c
index 6a8a5f5b0..bedda7fd5 100644
--- a/src/module.c
+++ b/src/module.c
@@ -580,7 +580,7 @@ void autoMemoryFreed(RedisModuleCtx *ctx, int type, void *ptr) {
if (!(ctx->flags & REDISMODULE_CTX_AUTO_MEMORY)) return;
int j;
- for (j = 0; j < ctx->amqueue_used; j++) {
+ for (j = ctx->amqueue_used - 1; j >= 0; j--) {
if (ctx->amqueue[j].type == type &&
ctx->amqueue[j].ptr == ptr)
{
@@ -588,6 +588,9 @@ void autoMemoryFreed(RedisModuleCtx *ctx, int type, void *ptr) {
/* Optimization: if this is the last element, we can
* reuse it. */
if (j == ctx->amqueue_used-1) ctx->amqueue_used--;
+
+ break;
+
}
}
}