caleb-brown/redict
1
0
Fork 0
mirror of https://codeberg.org/redict/redict.git synced 2025-01-22 16:18:28 -05:00
Code Issues Actions Packages Projects Releases Wiki Activity

Commit change in autoMemoryFreed(): first -> last.

Browse Source 
It's more natural to call the last entry added as "last", the original
commet got me confused until I actually read the code.
This commit is contained in:
antirez 2016-06-23 09:38:30 +02:00
parent f2dbc02f65
commit b507289750
1 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
src/module.c
View File

@ -621,11 +621,12 @@ void autoMemoryFreed(RedisModuleCtx *ctx, int type, void *ptr) {
{
ctx->amqueue[i].type = REDISMODULE_AM_FREED;
/* Switch the freed element and the top element, to avoid growing
/* Switch the freed element and the last element, to avoid growing
* the queue unnecessarily if we allocate/free in a loop */
if (i != ctx->amqueue_used-1) {
ctx->amqueue[i] = ctx->amqueue[ctx->amqueue_used-1];
}
/* Reduce the size of the queue because we either moved the top
* element elsewhere or freed it */
ctx->amqueue_used--;