|
From: Nicholas N. <nj...@ca...> - 2004-01-21 15:08:36
|
CVS commit by nethercote:
Convert "skin" to "tool" in various places; almost entirely within comments,
nothing that will affect code.
M +1 -1 AUTHORS 1.6
M +1 -1 cachegrind/cg_main.c 1.63
M +9 -9 coregrind/vg_errcontext.c 1.50
M +3 -3 coregrind/vg_malloc2.c 1.18
M +1 -1 coregrind/vg_mylibc.c 1.65
M +2 -2 coregrind/vg_needs.c 1.15
M +1 -1 coregrind/vg_replace_malloc.c 1.18
M +4 -4 coregrind/vg_scheduler.c 1.138
M +4 -4 coregrind/vg_signals.c 1.58
M +3 -3 coregrind/vg_symtab2.c 1.74
M +1 -1 coregrind/vg_symtypes.c 1.5
M +2 -2 coregrind/vg_to_ucode.c 1.124
M +3 -3 include/valgrind.h 1.25
M +3 -3 include/vg_profile.c 1.11
M +1 -1 memcheck/mac_malloc_wrappers.c 1.10
M +3 -3 memcheck/mac_shared.h 1.17
M +1 -1 memcheck/memcheck.h 1.18
M +1 -1 tests/filter_stderr_basic 1.15
--- valgrind/AUTHORS #1.5:1.6
@@ -3,5 +3,5 @@
Nicholas Nethercote, nj...@ca..., did the core/tool
-generalisation, and wrote Cachegrind and some of the other skins.
+generalisation, and wrote Cachegrind and some of the other tools.
Jeremy Fitzhardinge, je...@go..., wrote much of Helgrind, and lots
--- valgrind/cachegrind/cg_main.c #1.62:1.63
@@ -70,5 +70,5 @@ typedef
VgpCacheResults
}
- VgpSkinCC;
+ VgpToolCC;
/*------------------------------------------------------------*/
--- valgrind/coregrind/vg_errcontext.c #1.49:1.50
@@ -421,5 +421,5 @@ void VG_(maybe_record_error) ( ThreadId
/* Second top-level entry point to the error management subsystem, for
- errors that the skin want to report immediately, eg. because they're
+ errors that the tool wants to report immediately, eg. because they're
guaranteed to only happen once. This avoids all the recording and
comparing stuff. But they can be suppressed; returns True if it is
@@ -628,7 +628,7 @@ static Bool setLocationTy ( Char** p_cal
-/* Look for "skin" in a string like "skin1,skin2,skin3" */
+/* Look for "tool" in a string like "tool1,tool2,tool3" */
static __inline__
-Bool skin_name_present(Char *name, Char *names)
+Bool tool_name_present(Char *name, Char *names)
{
Bool found;
@@ -654,5 +654,5 @@ static void load_one_suppressions_file (
Bool eof;
Char buf[N_BUF+1];
- Char* skin_names;
+ Char* tool_names;
Char* supp_name;
@@ -665,5 +665,5 @@ static void load_one_suppressions_file (
while (True) {
- /* Assign and initialise the two suppression halves (core and skin) */
+ /* Assign and initialise the two suppression halves (core and tool) */
Supp* supp;
supp = VG_(arena_malloc)(VG_AR_CORE, sizeof(Supp));
@@ -694,9 +694,9 @@ static void load_one_suppressions_file (
buf[i] = '\0'; /* Replace ':', splitting into two strings */
- skin_names = & buf[0];
+ tool_names = & buf[0];
supp_name = & buf[i+1];
/* Is it a core suppression? */
- if (VG_(needs).core_errors && skin_name_present("core", skin_names))
+ if (VG_(needs).core_errors && tool_name_present("core", tool_names))
{
if (VG_STREQ(supp_name, "PThread"))
@@ -706,7 +706,7 @@ static void load_one_suppressions_file (
}
- /* Is it a skin suppression? */
+ /* Is it a tool suppression? */
else if (VG_(needs).skin_errors &&
- skin_name_present(VG_(details).name, skin_names))
+ tool_name_present(VG_(details).name, tool_names))
{
if (SK_(recognised_suppression)(supp_name, supp))
--- valgrind/coregrind/vg_malloc2.c #1.17:1.18
@@ -255,5 +255,5 @@ void ensure_mm_init ( void )
/* Use a checked red zone size of 1 word for our internal stuff,
and an unchecked zone of arbitrary size for the client. Of
- course the client's red zone can be checked by the skin, eg.
+ course the client's red zone can be checked by the tool, eg.
by using addressibility maps, but not by the mechanism implemented
here, which merely checks at the time of freeing that the red
@@ -262,5 +262,5 @@ void ensure_mm_init ( void )
arena_init ( &vg_arena[VG_AR_CORE], "core", 1, True, 262144, False );
- arena_init ( &vg_arena[VG_AR_SKIN], "skin", 1, True, 262144, False );
+ arena_init ( &vg_arena[VG_AR_SKIN], "tool", 1, True, 262144, False );
arena_init ( &vg_arena[VG_AR_SYMTAB], "symtab", 1, True, 262144, False );
@@ -1359,5 +1359,5 @@ void* VG_(arena_realloc) ( ArenaId aid,
/*------------------------------------------------------------*/
-/* All just wrappers to avoid exposing arenas to skins */
+/* All just wrappers to avoid exposing arenas to tools */
void* VG_(malloc) ( Int nbytes )
--- valgrind/coregrind/vg_mylibc.c #1.64:1.65
@@ -1035,5 +1035,5 @@ __inline__ Char* VG_(arena_strdup) ( Are
}
-/* Wrapper to avoid exposing skins to ArenaId's */
+/* Wrapper to avoid exposing tools to ArenaId's */
Char* VG_(strdup) ( const Char* s )
{
--- valgrind/coregrind/vg_needs.c #1.14:1.15
@@ -1,5 +1,5 @@
/*--------------------------------------------------------------------*/
-/*--- Stuff relating to skin data structures. ---*/
+/*--- Stuff relating to tool data structures. ---*/
/*--- vg_needs.c ---*/
/*--------------------------------------------------------------------*/
@@ -34,5 +34,5 @@
/* ---------------------------------------------------------------------
- Skin data structure initialisation
+ Tool data structure initialisation
------------------------------------------------------------------ */
--- valgrind/coregrind/vg_replace_malloc.c #1.17:1.18
@@ -37,5 +37,5 @@
be called from within Valgrind.
- This file can be #included into a skin that wishes to know about
+ This file can be #included into a tool that wishes to know about
calls to malloc(). It should define functions SK_(malloc) et al
that will be called.
--- valgrind/coregrind/vg_scheduler.c #1.137:1.138
@@ -1115,5 +1115,5 @@ VgSchedReturnCode VG_(scheduler) ( void
VG_(exitcode) = VG_(threads)[tid].m_ebx; /* syscall arg1 */
- /* Only run __libc_freeres if the skin says it's ok and
+ /* Only run __libc_freeres if the tool says it's ok and
it hasn't been overridden with --run-libc-freeres=no
on the command line. */
@@ -2962,8 +2962,8 @@ void do_client_request ( ThreadId tid )
}
- /* Note: for skins that replace malloc() et al, we want to call
+ /* Note: for tools that replace malloc() et al, we want to call
the replacement versions. For those that don't, we want to call
VG_(cli_malloc)() et al. We do this by calling SK_(malloc)(), which
- malloc-replacing skins must replace, but have its default definition
+ malloc-replacing tools must replace, but have its default definition
call */
@@ -3281,5 +3281,5 @@ void do_client_request ( ThreadId tid )
if (!whined) {
- // Allow for requests in core, but defined by skins, which
+ // Allow for requests in core, but defined by tools, which
// have 0 and 0 in their two high bytes.
Char c1 = (arg[0] >> 24) & 0xff;
--- valgrind/coregrind/vg_signals.c #1.57:1.58
@@ -985,5 +985,5 @@ void vg_push_signal_frame ( ThreadId tid
sigNo, signame(sigNo), tid );
- /* Signal delivery to skins */
+ /* Signal delivery to tools */
VG_TRACK( pre_deliver_signal, tid, sigNo, /*alt_stack*/False );
@@ -991,5 +991,5 @@ void vg_push_signal_frame ( ThreadId tid
esp_top_of_frame = tst->m_esp;
- /* Signal delivery to skins */
+ /* Signal delivery to tools */
VG_TRACK( pre_deliver_signal, tid, sigNo, /*alt_stack*/True );
}
@@ -1171,5 +1171,5 @@ Int vg_pop_signal_frame ( ThreadId tid )
VG_(proxy_setsigmask)(tid);
- /* Notify skins */
+ /* Notify tools */
VG_TRACK( post_deliver_signal, tid, sigNo );
@@ -1686,5 +1686,5 @@ void vg_sync_signalhandler ( Int sigNo,
using some memory which had not previously been used.
This catches those faults, makes the memory accessible,
- and calls the skin to initialize that page.
+ and calls the tool to initialize that page.
*/
static Int recursion = 0;
--- valgrind/coregrind/vg_symtab2.c #1.73:1.74
@@ -1545,5 +1545,5 @@ Bool get_fnname ( Bool demangle, Addr a,
}
-/* This is available to skins... always demangle C++ names,
+/* This is available to tools... always demangle C++ names,
match anywhere in function, but don't show offsets. */
Bool VG_(get_fnname) ( Addr a, Char* buf, Int nbuf )
@@ -1554,5 +1554,5 @@ Bool VG_(get_fnname) ( Addr a, Char* buf
}
-/* This is available to skins... always demangle C++ names,
+/* This is available to tools... always demangle C++ names,
match anywhere in function, and show offset if nonzero. */
Bool VG_(get_fnname_w_offset) ( Addr a, Char* buf, Int nbuf )
@@ -1563,5 +1563,5 @@ Bool VG_(get_fnname_w_offset) ( Addr a,
}
-/* This is available to skins... always demangle C++ names,
+/* This is available to tools... always demangle C++ names,
only succeed if 'a' matches first instruction of function,
and don't show offsets. */
--- valgrind/coregrind/vg_symtypes.c #1.4:1.5
@@ -690,5 +690,5 @@ Char *VG_(describe_addr)(ThreadId tid, A
bufsz *= 2;
- /* use skin malloc so that the skin client can free it */
+ /* use tool malloc so that the skin client can free it */
n = VG_(malloc)(bufsz);
if (buf != NULL && bufidx != 0)
--- valgrind/coregrind/vg_to_ucode.c #1.123:1.124
@@ -7173,6 +7173,6 @@ Int VG_(disBB) ( UCodeBlock* cb, Addr ei
* into the `extra4b' field of the basic-block-ending JMP.
*
- * The INCEIPs and JMP.extra4b fields allows a skin to track x86
- * instruction sizes, important for some skins (eg. cache simulation).
+ * The INCEIPs and JMP.extra4b fields allows a tool to track x86
+ * instruction sizes, important for some tools (eg. Cachegrind).
*/
if (VG_(clo_single_step)) {
--- valgrind/include/valgrind.h #1.24:1.25
@@ -160,5 +160,5 @@ typedef
VG_USERREQ__COUNT_ERRORS = 0x1201,
- /* These are useful and can be interpreted by any skin that tracks
+ /* These are useful and can be interpreted by any tool that tracks
malloc() et al, by using vg_replace_malloc.c. */
VG_USERREQ__MALLOCLIKE_BLOCK = 0x1301,
@@ -274,6 +274,6 @@ VALGRIND_PRINTF_BACKTRACE(const char *fo
-/* Counts the number of errors that have been recorded by a skin. Nb:
- the skin must record the errors with VG_(maybe_record_error)() or
+/* Counts the number of errors that have been recorded by a tool. Nb:
+ the tool must record the errors with VG_(maybe_record_error)() or
VG_(unique_error)() for them to be counted. */
#define VALGRIND_COUNT_ERRORS \
--- valgrind/include/vg_profile.c #1.10:1.11
@@ -1,6 +1,6 @@
/*--------------------------------------------------------------------*/
-/*--- Profiling machinery. #include this file into a skin to ---*/
-/*--- enable --profile=yes, but not for release versions of skins, ---*/
+/*--- Profiling machinery. #include this file into a tool to ---*/
+/*--- enable --profile=yes, but not for release versions of tools, ---*/
/*--- because it uses glibc code. ---*/
/*--- vg_profile.c ---*/
@@ -77,5 +77,5 @@ void VGP_(register_profile_event) ( Int
VG_(printf)("\nProfile event #%d being registered as `%s'\n"
"already registered as `%s'.\n"
- "Note that skin and core event numbers must not overlap.\n",
+ "Note that tool and core event numbers must not overlap.\n",
n, name, vgp_names[n]);
VG_(skin_panic)("profile event already registered");
--- valgrind/memcheck/mac_malloc_wrappers.c #1.9:1.10
@@ -45,5 +45,5 @@ static UInt cmalloc_bs_mallocd = 0;
UInt VG_(vg_malloc_redzone_szB) = 16;
-/* Function pointers for the two skins to track interesting events. */
+/* Function pointers for the two tools to track interesting events. */
void (*MAC_(new_mem_heap)) ( Addr a, UInt len, Bool is_inited ) = NULL;
void (*MAC_(ban_mem_heap)) ( Addr a, UInt len ) = NULL;
--- valgrind/memcheck/mac_shared.h #1.16:1.17
@@ -155,5 +155,5 @@ typedef
/*------------------------------------------------------------*/
-/*--- Profiling of skins and memory events ---*/
+/*--- Profiling of tools and memory events ---*/
/*------------------------------------------------------------*/
@@ -164,5 +164,5 @@ typedef
VgpESPAdj
}
- VgpSkinCC;
+ VgpToolCC;
/* Define to collect detailed performance info. */
@@ -271,5 +271,5 @@ extern void MAC_(print_common_debug_usag
extern VgHashTable MAC_(malloc_list);
-/* Function pointers for the two skins to track interesting events. */
+/* Function pointers for the two tools to track interesting events. */
extern void (*MAC_(new_mem_heap)) ( Addr a, UInt len, Bool is_inited );
extern void (*MAC_(ban_mem_heap)) ( Addr a, UInt len );
--- valgrind/memcheck/memcheck.h #1.17:1.18
@@ -83,5 +83,5 @@ typedef
/* These two have been moved into core, because they are useful for
- any skin that tracks heap blocks. Hence the suffix. But they're
+ any tool that tracks heap blocks. Hence the suffix. But they're
still here for backwards compatibility, although Valgrind will
abort with an explanatory message if you use them. */
--- valgrind/tests/filter_stderr_basic #1.14:1.15
@@ -8,5 +8,5 @@
# Remove "<name>, a <description> for x86-linux." line and the following
-# copyright notice line. Works for skin and core intro lines.
+# copyright notice line. Works for tool and core intro lines.
sed "/^.*, .* for x86-linux\./ , /./ d" |
|