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