Benjamin Wolsey wrote:
There is code to support several mallocs in libbase. Old code for using
dlmalloc is in libbase/dlmalloc.{cpp,h} and utility.h. There also used
to be special support for dmalloc (detection in configure.ac and some
ifdefs in utility.cpp).
A long time ago I added dmalloc for memory debugging, but these days
I think using valgrind works better as nothing has to be recompiled. It
can be depreciated now.
Because this code seems obsolete now we have jemalloc, which should work
well cross-platform (besides any standard allocator), I propose to drop
it.
I've had very good luck with jemalloc, to the point I think we should
consider making it the default.
As a side effect, dump_memory_stats() in utility.cpp would go. A comment
suggests this never worked for dmalloc, but the code cleverly disables
it unless dmalloc is being used. Rob has already added newer code in
libbase for checking memory usage, so dump_memory_stats may also be
obsolete.
Yes, there is much better code that uses mallinfo() for memory
statistics, plus jemalloc has a more verbose version as well.
- rob -
_______________________________________________
Gnash-dev mailing list
[email protected]
http://lists.gnu.org/mailman/listinfo/gnash-dev
