Josh Stone <jistone(a)redhat.com> writes:
- /* First hash function: simply take the modul but prevent zero.
*/
- size_t idx = 1 + hval % htab->size;
+ /* First hash function: simply take the modul but prevent zero. Small values
+ * can skip the division, which helps performance when this is common. */
+ size_t idx = 1 + (hval < htab->size ? hval : hval % htab->size);
Again the continuation *. Otherwise this is fine, feel free to commit
after correcting this nit.
Thanks,
PM