Re: another hash cleanup

[Jim Meyering]

Eric Blake wrote:

> Jim Meyering <jim <at> meyering.net> writes:
>
>> >    if (new_table == NULL)
>> >      return false;
>> > +  if (new_table->n_buckets == table->n_buckets)
>> > +    return true;
>
> Aargh.  Ten minutes after I push, I finally see my memory leak.
>
> Obviously, new_table needs to be freed before returning true.

Hah!  I should have looked at more than the patch.

Is that code path already exercised by test-hash.c?
If so, I would have seen it eventually, via valgrind.