From: Martin Quinson <martin.quinson@loria.fr>
Date: Wed, 26 Jul 2017 19:53:58 +0000 (+0200)
Subject: fix tracing tests
X-Git-Tag: v3_17~318
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/5faf90176fb61aaa518448b708f13ba093432d26?ds=sidebyside

fix tracing tests

This will result in a leak in the tracing::value content, but this is
unavoidable as long as the constructor is not really initializing the
fields but instead initializing the ret field.
---

diff --git a/src/instr/instr_paje_types.cpp b/src/instr/instr_paje_types.cpp
index 389ff98d77..c93da968dc 100644
--- a/src/instr/instr_paje_types.cpp
+++ b/src/instr/instr_paje_types.cpp
@@ -52,7 +52,7 @@ void PJ_type_free (type_t type)
   xbt_dict_cursor_t cursor = nullptr;
   xbt_dict_foreach (type->values, cursor, value_name, val) {
     XBT_DEBUG("free value %s, child of %s", val->name, val->father->name);
-    delete val;
+    xbt_free(val);
   }
   xbt_dict_free (&type->values);
   xbt_free (type->name);
diff --git a/src/instr/instr_paje_values.cpp b/src/instr/instr_paje_values.cpp
index f46eed5dd2..84152ec3af 100644
--- a/src/instr/instr_paje_values.cpp
+++ b/src/instr/instr_paje_values.cpp
@@ -31,9 +31,11 @@ value::value(const char* name, const char* color, type_t father)
 
 value::~value()
 {
+  /* FIXME: this should be cleanable
   xbt_free(name);
   xbt_free(color);
   xbt_free(id);
+  */
 }
 
 value* value::get_or_new(const char* name, const char* color, type_t father)