Always output all leak kinds in a xtree leak result file.

- The option --xtree-leak=yes (to output leak result in xtree format)
  automatically activates the option --show-leak-kinds=all,
  as xtree visualisation tools such as kcachegrind can in any case
  select what kind of leak to visualise.

diff --git a/NEWS b/NEWS
index 8b6d4b361bdb763e079a1de651cd70c6cd61398c..c311665804cd4cf1b71dd9ee079afc539346493b 100644 (file)
--- a/NEWS
+++ b/NEWS
@@ -29,6 +29,12 @@ support for X86/macOS 10.13, AMD64/macOS 10.13.
   - The default value for --read-inline-info is now "yes" on
     Linux/Android/Solaris. It is still "no" on other OS.
 
+* Memcheck:
+  - The option --xtree-leak=yes (to output leak result in xtree format)
+    automatically activates the option --show-leak-kinds=all,
+    as xtree visualisation tools such as kcachegrind can in any case
+    select what kind of leak to visualise.
+
 * ==================== OTHER CHANGES ====================
 
 

diff --git a/memcheck/docs/mc-manual.xml b/memcheck/docs/mc-manual.xml
index eb1dfab272e5f7dc3816f0d25b92f880c347dd4d..a993479a9c2edc43a44402ee31b5716d69d0abe4 100644 (file)
--- a/memcheck/docs/mc-manual.xml
+++ b/memcheck/docs/mc-manual.xml
@@ -888,9 +888,11 @@ is <option>--errors-for-leak-kinds=definite,possible</option>
     <listitem>
       <para>If set to yes, the results for the leak search done at exit will be
         output in a 'Callgrind Format' execution tree file. Note that this
-        automatically sets the option <option>--leak-check=full</option>.
-        The produced file
-       will contain the following events:</para>
+        automatically sets the options <option>--leak-check=full</option>
+        and <option>--show-leak-kinds=all</option>, to allow
+        xtree visualisation tools such as kcachegrind to select what kind
+        to leak to visualise.
+        The produced file will contain the following events:</para>
       <itemizedlist>
         <listitem><para><option>RB</option> : Reachable Bytes</para></listitem> 
          <listitem><para><option>PB</option> : Possibly lost Bytes</para></listitem>

diff --git a/memcheck/mc_main.c b/memcheck/mc_main.c
index 9b2e654902f122a008e5acfc0c52682bcb7751c7..3ef7cb913803b2e165faf3f405dea24dd19052fd 100644 (file)
--- a/memcheck/mc_main.c
+++ b/memcheck/mc_main.c
@@ -7997,6 +7997,7 @@ static void mc_fini ( Int exitcode )
                                              MC_(clo_xtree_leak_file));
          lcp.xt_filename = xt_filename;
          lcp.mode = LC_Full;
+         lcp.show_leak_kinds = MC_(all_Reachedness)();
       }
       else
          lcp.xt_filename = NULL;