|
From: <sv...@va...> - 2005-08-24 23:08:28
|
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 ) \
|