Re: [Bug-gnulib] Patch proposal: 1-gary-safe-xfree.patch

[Jim Meyering]

Paul Eggert <address@hidden> wrote:
> "Gary V. Vaughan" <address@hidden> writes:
>
>> I propose applying your patch without autoconfigury...
>
> I thought about it a bit more, and decided that it'd be better to use
> a solution that simply causes 'free' to work as expected in C89 or
> letter.  We shouldn't introduce yet another symbol like 'xfree'; we
> should just make 'free' work.  So I propose the following patch instead.
>
> I'm also proposing changing the maintainer for the xalloc module to 'all'.
> (The actual maintainers have mostly been Bruno, Jim, and myself.)
>
> I'd still like to get rid of XMALLOC, XCALLOC, XREALLOC, XFREE, CCLONE,
> and CLONE, but that's a different matter, which I'll look into later.
>
> 2003-09-12  Paul Eggert  <address@hidden>
>
>       * modules/xalloc: Add m4/free-null.m4.  Change maintainer to "all".
>       * lib/xalloc.h (free) [!HAVE_FREE_NULL]: New macro.
>         (rpl_free) [!HAVE_FREE_NULL]: New function.
>         (XFREE): Use 'free'.  Add a comment that XFREE is obsolescent.
>         * m4/xalloc.m4 (gl_XALLOC): Require AC_C_INLINE, gl_FUNC_FREE_NULL.
>         * m4/free-null.m4: New file.

I like it.  Thanks!