Phillip Wood <phillip.wood123@xxxxxxxxx> writes: >> + } > check_int(count, ARRAY_SIZE(key_val)); > > to check that we iterate over all the entries as well as checking the > size of the hashmap here. I think check_int() macro wants the comparison operator in the middle, but other than that small typo, the suggestion sounds quite sensible. If the iterator does not yield anything, the current test would still pass. Thanks.