Modularised the malloc/free stuff into two modules:  m_mallocfree for the
malloc/free implementation, and m_replacemalloc with the stuff for the tools
that replace malloc with their own version.  Previously these two areas of
functionality were mixed up somewhat.




git-svn-id: svn://svn.valgrind.org/valgrind/trunk@3648 a5019735-40e9-0310-863c-91ae7b9d1cf9
diff --git a/coregrind/vg_pthreadmodel.c b/coregrind/vg_pthreadmodel.c
index 84f2dc5..a54c62b 100644
--- a/coregrind/vg_pthreadmodel.c
+++ b/coregrind/vg_pthreadmodel.c
@@ -207,9 +207,7 @@
    // TL_(malloc) being a weak symbol, and it just doesn't fit with the
    // VG_(tdict) approach that we've switched to.  The right way to do this
    // will be to provide a function in the core that checks if
-   // VG_(tdict).malloc_malloc has been set (or perhaps it would be better
-   // to check whether VG_USERREQ__GET_MALLOCFUNCS has been called, ie.
-   // whether replace_malloc.c:init() has been called);  if so, it should
+   // VG_(tdict).malloc_malloc has been set;  if so, it should
    // call it, if not, it should call VG_(cli_malloc)().
 //   VG_(tl_malloc_called_deliberately) = True;
 //   data = TL_(malloc)(sizeof(*data));