Shown as, https://github.com/redis/redis/blob/unstable/src/dict.c#L1056
for (j = 0; j < count; j++) {
if (dictIsRehashing(d))
_dictRehashStep(d);
else
break;
}
it may be better like below:
if (d->pauserehash == 0) dictRehash(d,count);
about the benefits:
- code style, more clear
- maybe better performance