[Valgrind-developers] valgrind: r4508 - branches/VALGRIND_3_0_BRANCH/memcheck

[<sv...@va...>]

Author: dirk
Date: 2005-08-25 00:08:24 +0100 (Thu, 25 Aug 2005)
New Revision: 4508

Log:
svn merge 4491: add comment

Modified:
   branches/VALGRIND_3_0_BRANCH/memcheck/mac_replace_strmem.c


Modified: branches/VALGRIND_3_0_BRANCH/memcheck/mac_replace_strmem.c
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D
--- branches/VALGRIND_3_0_BRANCH/memcheck/mac_replace_strmem.c	2005-08-24=
 23:08:12 UTC (rev 4507)
+++ branches/VALGRIND_3_0_BRANCH/memcheck/mac_replace_strmem.c	2005-08-24=
 23:08:24 UTC (rev 4508)
@@ -227,6 +227,10 @@
 STRNLEN(m_libc_so_6, strnlen)
   =20
=20
+// Note that this replacement often doesn't get used because gcc inlines
+// calls to strlen() with its own built-in version.  This can be very
+// confusing if you aren't expecting it.  Other small functions in this =
file
+// may also be inline by gcc.
 #define STRLEN(soname, fnname) \
    SizeT VG_REPLACE_FUNCTION(soname,fnname)( const char* str ); \
    SizeT VG_REPLACE_FUNCTION(soname,fnname)( const char* str ) \