]> git.ipfire.org Git - thirdparty/valgrind.git/commitdiff
Convert "skin" to "tool" in various places; almost entirely within comments,
authorNicholas Nethercote <n.nethercote@gmail.com>
Wed, 21 Jan 2004 15:08:04 +0000 (15:08 +0000)
committerNicholas Nethercote <n.nethercote@gmail.com>
Wed, 21 Jan 2004 15:08:04 +0000 (15:08 +0000)
nothing that will affect code.

git-svn-id: svn://svn.valgrind.org/valgrind/trunk@2217

18 files changed:
AUTHORS
cachegrind/cg_main.c
coregrind/vg_errcontext.c
coregrind/vg_malloc2.c
coregrind/vg_mylibc.c
coregrind/vg_needs.c
coregrind/vg_replace_malloc.c
coregrind/vg_scheduler.c
coregrind/vg_signals.c
coregrind/vg_symtab2.c
coregrind/vg_symtypes.c
coregrind/vg_to_ucode.c
include/valgrind.h
include/vg_profile.c
memcheck/mac_malloc_wrappers.c
memcheck/mac_shared.h
memcheck/memcheck.h
tests/filter_stderr_basic

diff --git a/AUTHORS b/AUTHORS
index 4165b09386e548fd7911102e3d7e1a0f5e9bf4e6..d9f5670fdf595beaffd6c7ecd5b1ccd4f8c1d438 100644 (file)
--- a/AUTHORS
+++ b/AUTHORS
@@ -2,7 +2,7 @@
 Julian Seward, jseward@acm.org, is the main author.
 
 Nicholas Nethercote, njn25@cam.ac.uk, 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, jeremy@goop.org, wrote much of Helgrind, and lots
 of low-level syscall/signal stuff.
index 4726c907e1e300f04f26e696831ad091adeb6c34..54850a957f17047c883e5ff79126b21a72f17e81 100644 (file)
@@ -69,7 +69,7 @@ typedef
       VgpCacheSimulate,
       VgpCacheResults
    } 
-   VgpSkinCC;
+   VgpToolCC;
 
 /*------------------------------------------------------------*/
 /*--- Output file related stuff                            ---*/
index 892502ff7d55af2e2a8c9bc0235a74d135862bd2..dffe4f3e1cac9bc678ad2dd4c688d0fa816b13e2 100644 (file)
@@ -420,7 +420,7 @@ void VG_(maybe_record_error) ( ThreadId tid,
 }
 
 /* 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
    suppressed.  Bool `print_error' dictates whether to print the error. 
@@ -627,9 +627,9 @@ static Bool setLocationTy ( Char** p_caller, SuppLocTy* p_ty )
 }
 
 
-/* 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;
    Char *s = NULL;   /* Shut gcc up */
@@ -653,7 +653,7 @@ static void load_one_suppressions_file ( Char* filename )
    Int   fd, i;
    Bool  eof;
    Char  buf[N_BUF+1];
-   Char* skin_names;
+   Char* tool_names;
    Char* supp_name;
 
    fd = VG_(open)( filename, VKI_O_RDONLY, 0 );
@@ -664,7 +664,7 @@ static void load_one_suppressions_file ( Char* filename )
    }
 
    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));
       supp->count = 0;
@@ -693,11 +693,11 @@ static void load_one_suppressions_file ( Char* filename )
       }
       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"))
             supp->skind = PThreadSupp;
@@ -705,9 +705,9 @@ static void load_one_suppressions_file ( Char* filename )
             goto syntax_error;
       }
 
-      /* 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)) 
          {
index 6aa035c907dc802c864e9d51e4e59f79e11fcd43..f1714805880ca81a2d0420b0518a15959a7e00c6 100644 (file)
@@ -254,14 +254,14 @@ 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 
       zone words are unchanged. */
 
    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 );
 
@@ -1358,7 +1358,7 @@ void* VG_(arena_realloc) ( ArenaId aid, void* ptr,
 /*--- Skin-visible functions.                              ---*/
 /*------------------------------------------------------------*/
 
-/* All just wrappers to avoid exposing arenas to skins */
+/* All just wrappers to avoid exposing arenas to tools */
 
 void* VG_(malloc) ( Int nbytes )
 {
index 99200aa5126bc2a22e70e13d0da2957121962d02..41cd3ee9b586fe0aa345dea2c30f32cce1507fdd 100644 (file)
@@ -1034,7 +1034,7 @@ __inline__ Char* VG_(arena_strdup) ( ArenaId aid, const Char* s )
    return res;
 }
 
-/* Wrapper to avoid exposing skins to ArenaId's */
+/* Wrapper to avoid exposing tools to ArenaId's */
 Char* VG_(strdup) ( const Char* s )
 {
    return VG_(arena_strdup) ( VG_AR_SKIN, s ); 
index baad2a32bf5e105f39c4f8da55d233f822dc582a..da4bc9cfee647675d214a4e307be9dd0f94cd889 100644 (file)
@@ -1,6 +1,6 @@
 
 /*--------------------------------------------------------------------*/
-/*--- Stuff relating to skin data structures.                      ---*/
+/*--- Stuff relating to tool data structures.                      ---*/
 /*---                                                   vg_needs.c ---*/
 /*--------------------------------------------------------------------*/
 
@@ -33,7 +33,7 @@
 
 
 /* ---------------------------------------------------------------------
-   Skin data structure initialisation
+   Tool data structure initialisation
    ------------------------------------------------------------------ */
 
 /* Init with default values. */
index 9c310dd309a5021a22091619281190d781ef3a6b..ce02c9790d4580f54208eb2155f6b4941bf863d5 100644 (file)
@@ -36,7 +36,7 @@
    have no prototypes in vg_include.h, since they are not intended to
    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.
    ------------------------------------------------------------------ */
index 850f6685456458232141d3c7e585e3b0813661cf..ce04d4d27e064317ea89b0ae8905f8d032792cc7 100644 (file)
@@ -1114,7 +1114,7 @@ VgSchedReturnCode VG_(scheduler) ( void )
                /* If __NR_exit, remember the supplied argument. */
                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. */
 
@@ -2961,10 +2961,10 @@ void do_client_request ( ThreadId tid )
          break;
       }
 
-      /* 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 */
 
       /* Note: for MALLOC and FREE, must set the appropriate "lock"... see
@@ -3280,7 +3280,7 @@ void do_client_request ( ThreadId tid )
            static Bool whined = False;
 
            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;
                Char c2 = (arg[0] >> 16) & 0xff;
index c5605646bf511ba90cd4d2477409cc6a13df4b9a..79ca5aaf2aa757de1a8d09c439ba5c1ce5d11c01 100644 (file)
@@ -984,13 +984,13 @@ void vg_push_signal_frame ( ThreadId tid, const vki_ksiginfo_t *siginfo )
             "delivering signal %d (%s) to thread %d: on ALT STACK", 
             sigNo, signame(sigNo), tid );
 
-      /* Signal delivery to skins */
+      /* Signal delivery to tools */
       VG_TRACK( pre_deliver_signal, tid, sigNo, /*alt_stack*/False );
       
    } else {
       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 );
    }
 
@@ -1170,7 +1170,7 @@ Int vg_pop_signal_frame ( ThreadId tid )
    tst->sig_mask  = frame->mask;
    VG_(proxy_setsigmask)(tid);
 
-   /* Notify skins */
+   /* Notify tools */
    VG_TRACK( post_deliver_signal, tid, sigNo );
 
    return sigNo;
@@ -1685,7 +1685,7 @@ void vg_sync_signalhandler ( Int sigNo, vki_ksiginfo_t *info, struct vki_ucontex
            is a permissions fault, then it means that the client is
            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;
 
index c779873760a02c5853268b90220a938cd3291d27..3c15da5f951081f9e1ef46b8f4589f341c9ecd94 100644 (file)
@@ -1544,7 +1544,7 @@ Bool get_fnname ( Bool demangle, Addr a, Char* buf, Int nbuf,
    return True;
 }
 
-/* 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 )
 {
@@ -1553,7 +1553,7 @@ Bool VG_(get_fnname) ( Addr a, Char* buf, Int nbuf )
                        /*show offset?*/False );
 }
 
-/* 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 )
 {
@@ -1562,7 +1562,7 @@ Bool VG_(get_fnname_w_offset) ( Addr a, Char* buf, Int nbuf )
                        /*show offset?*/True );
 }
 
-/* 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. */
 Bool VG_(get_fnname_if_entry) ( Addr a, Char* buf, Int nbuf )
index 2ce4033744a2f0e7ed41f4e0f081d5c8b823cef0..f291d16717f5e2926aa6cc85ab8a62fd6089b8fb 100644 (file)
@@ -689,7 +689,7 @@ Char *VG_(describe_addr)(ThreadId tid, Addr addr)
         else
            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)
            VG_(memcpy)(n, buf, bufidx);
index cdd25e75f3c699dd7d3df77428373c4ace02fde8..af83df0cc8ec277b2fe7b39ad3c1ab78b78b8c7d 100644 (file)
@@ -7172,8 +7172,8 @@ Int VG_(disBB) ( UCodeBlock* cb, Addr eip0 )
     * in the basic block.  For them we patch in the x86 instruction size 
     * 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)) {
       eip = disInstr ( cb, eip, &isEnd );
index 52a4f37a2c1edc0b5342959baeeb0954edfaf138..8cd07c9ebac4cad882c17a7d7dbd3e921aa4cc85 100644 (file)
@@ -159,7 +159,7 @@ typedef
              Valgrind's output to /dev/null and still count errors. */
           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,
           VG_USERREQ__FREELIKE_BLOCK   = 0x1302,
@@ -273,8 +273,8 @@ VALGRIND_PRINTF_BACKTRACE(const char *format, ...)
    })
 
 
-/* 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                                           \
    ({unsigned int _qyy_res;                                             \
index cc732e166fd7fe60134500fc57c9682909a11da4..877efadaf3e9b42d272920e202873e1277cebbd3 100644 (file)
@@ -1,7 +1,7 @@
 
 /*--------------------------------------------------------------------*/
-/*--- 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 ---*/
 /*--------------------------------------------------------------------*/
@@ -76,7 +76,7 @@ void VGP_(register_profile_event) ( Int n, Char* name )
    if (vgp_names[n] != NULL) {
       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");
    }
index 8714bb8ff2c02d367df6b2f0b270239675ebbdcf..b3c5353287961e418cf1d417ac1da548ae4afcf9 100644 (file)
@@ -44,7 +44,7 @@ static UInt cmalloc_bs_mallocd = 0;
 /* We want a 16B redzone on heap blocks for Addrcheck and Memcheck */
 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;
 void (*MAC_(die_mem_heap)) ( Addr a, UInt len )                  = NULL;
index f36175ce6b4501317c51305bde3a33293dfc7d96..c968a0820989bd5689042d8d9bf81799ce85bc16 100644 (file)
@@ -154,7 +154,7 @@ typedef
    MAC_Chunk;
 
 /*------------------------------------------------------------*/
-/*--- Profiling of skins and memory events                 ---*/
+/*--- Profiling of tools and memory events                 ---*/
 /*------------------------------------------------------------*/
 
 typedef 
@@ -163,7 +163,7 @@ typedef
       VgpSetMem,
       VgpESPAdj
    } 
-   VgpSkinCC;
+   VgpToolCC;
 
 /* Define to collect detailed performance info. */
 /* #define MAC_PROFILE_MEMORY */
@@ -270,7 +270,7 @@ extern void MAC_(print_common_debug_usage)       ( void );
 /* For tracking malloc'd blocks */
 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 );
 extern void (*MAC_(die_mem_heap)) ( Addr a, UInt len );
index e817c1377e564f0e8a7173a77fc6a6c973a7759a..03d7ff9754ef26ecadf6c1a57afe3df2283401b2 100644 (file)
@@ -82,7 +82,7 @@ typedef
       VG_USERREQ__COUNT_LEAKS,
 
       /* 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. */
       VG_USERREQ__MALLOCLIKE_BLOCK__OLD_DO_NOT_USE,
index 136db5170a86f3aef39269bcdca7c3ab1bf1e123..1150f11721a2eb6e01706bb4018e7c464bc2503f 100755 (executable)
@@ -7,7 +7,7 @@
 sed "s/\(==\|--\|\+\+\|\*\*\)[0-9]\{1,5\}\1 //"             |
 
 # 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"                                | 
 
 # Remove other introductory lines