(no subject)

[Bob Friesenhahn]

On Wed, 31 Mar 2004, Gary V. Vaughan wrote:

> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
>
> Okay to commit?
>
> This addresses the memory management problems in libltdl brought
> to light by the kaffe people.

The style of this change confuses me, and it doesn't seem to do
exactly the same thing:

   /* If ARGZ_LEN has shrunk to nothing, release ARGZ's memory.  */
   if (!argz_len)
-    LT_DLFREE (argz);
+    argz = (free (argz), (char *) 0);

The change of lt_ functions to 'x' versions (e.g. xmalloc)
implies that it is intended that these functions be moved out of
ltdl.c and into another source module.  If so, the 'x' versions may
then conflict with other 'x' versions used by the dependent program.
It seems better to retain the lt_ prefix because then the origin of
the functions is more clear and there is no danger of conflict.

Bob
======================================
Bob Friesenhahn
address@hidden
http://www.simplesystems.org/users/bfriesen