[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: another hash cleanup
From: |
Jim Meyering |
Subject: |
Re: another hash cleanup |
Date: |
Fri, 19 Jun 2009 16:06:21 +0200 |
Eric Blake wrote:
...
> Done. And with that on master, I've rebased my hash_rehash memory leak plug:
>
> http://repo.or.cz/w/gnulib/ericb.git?a=shortlog;h=refs/heads/hash
> $ git pull git://repo.or.cz/gnulib/ericb.git hash
Thanks. I've just started looking, and one thing struck me:
+static bool
+transfer_entries (Hash_table *src, Hash_table *dst, bool safe)
+{
I'd prefer to make this like memcpy, strcpy, etc.
which have the destination object on the left:
transfer_entries (Hash_table *dst, Hash_table *src, bool safe)
- Re: another hash cleanup, (continued)
- Re: another hash cleanup, Jim Meyering, 2009/06/18
- Re: another hash cleanup, Eric Blake, 2009/06/18
- Re: another hash cleanup, Jim Meyering, 2009/06/19
- Re: another hash cleanup, Eric Blake, 2009/06/19
- Re: another hash cleanup, Jim Meyering, 2009/06/19
- Re: another hash cleanup, Eric Blake, 2009/06/19
- Re: another hash cleanup, Jim Meyering, 2009/06/19
- Re: another hash cleanup, Eric Blake, 2009/06/19
- Re: another hash cleanup, Jim Meyering, 2009/06/19
- Re: another hash cleanup, Eric Blake, 2009/06/19
- Re: another hash cleanup,
Jim Meyering <=
- Re: another hash cleanup, Jim Meyering, 2009/06/19