Hi Christoph, * Christoph Egger wrote on Wed, Aug 31, 2005 at 11:59:29AM CEST: > > The attached patch touches libltdl/config/ltmain.m4sh > > The program_name is a const char * per declaration. > base_name() returns a const char *. > There's no need make a duplicate. The duplicate just > causes an unnecessary memory leak.
Oh well. Somebody else came in and suggested that change to avoid a compiler warning. You suggest it to avoid a constant-size memleak in a programs whose sole purpose is to call exec*(). Both concerns are bogus to some extent. No nontrivial program compiles free of warnings with all compilers on earth, and no systems Libtool runs on does not have a decent kernel with virtual memory management that reclaims memory of dead processes. Given equal options, I believe we should just leave things as they are. Cheers, Ralf
