New: s4u::Host.execute() for remote executions

[simgrid.git] / src / instr / instr_paje_types.cpp

diff --git a/src/instr/instr_paje_types.cpp b/src/instr/instr_paje_types.cpp
index 700a898..73ce50e 100644 (file)
--- a/src/instr/instr_paje_types.cpp
+++ b/src/instr/instr_paje_types.cpp
@@ -22,21 +22,18 @@ simgrid::instr::Type* PJ_type_get_root()
 
 simgrid::instr::Type::Type(const char* typeNameBuff, const char* key, const char* color, e_entity_types kind,
                            Type* father)
+    : kind_(kind), father_(father)
 {
   if (typeNameBuff == nullptr || key == nullptr){
     THROWF(tracing_error, 0, "can't create a new type with name or key equal nullptr");
   }
 
   this->name_     = xbt_strdup(typeNameBuff);
-  this->father_   = father;
-  this->kind_     = kind;
   this->children_ = xbt_dict_new_homogeneous(nullptr);
   this->values_   = xbt_dict_new_homogeneous(nullptr);
   this->color_    = xbt_strdup(color);
 
-  char str_id[INSTR_DEFAULT_STR_SIZE];
-  snprintf (str_id, INSTR_DEFAULT_STR_SIZE, "%lld", instr_new_paje_id());
-  this->id_ = xbt_strdup(str_id);
+  this->id_ = bprintf("%lld", instr_new_paje_id());
 
   if (father != nullptr){
     xbt_dict_set(father->children_, key, this, nullptr);
@@ -58,8 +55,7 @@ void PJ_type_free(simgrid::instr::Type* type)
   xbt_free(type->id_);
   xbt_free(type->color_);
   xbt_dict_free(&type->children_);
-  xbt_free (type);
-  type = nullptr;
+  delete type;
 }
 
 void recursiveDestroyType(simgrid::instr::Type* type)
@@ -116,7 +112,7 @@ simgrid::instr::Type* simgrid::instr::Type::containerNew(const char* name, simgr
     rootType = ret;
   } else {
     XBT_DEBUG("ContainerType %s(%s), child of %s(%s)", ret->name_, ret->id_, father->name_, father->id_);
-    DefineContainerEvent(ret);
+    LogContainerTypeDefinition(ret);
   }
   return ret;
 }
@@ -159,11 +155,9 @@ simgrid::instr::Type* simgrid::instr::Type::linkNew(const char* name, Type* fath
     THROWF (tracing_error, 0, "can't create a link type with a nullptr name");
   }
 
-  Type* ret = nullptr;
-
   char key[INSTR_DEFAULT_STR_SIZE];
   snprintf(key, INSTR_DEFAULT_STR_SIZE, "%s-%s-%s", name, source->id_, dest->id_);
-  ret = new Type (name, key, nullptr, TYPE_LINK, father);
+  Type* ret = new Type(name, key, nullptr, TYPE_LINK, father);
   XBT_DEBUG("LinkType %s(%s), child of %s(%s)  %s(%s)->%s(%s)", ret->name_, ret->id_, father->name_, father->id_,
             source->name_, source->id_, dest->name_, dest->id_);
   LogLinkTypeDefinition(ret, source, dest);
@@ -176,9 +170,7 @@ simgrid::instr::Type* simgrid::instr::Type::stateNew(const char* name, Type* fat
     THROWF (tracing_error, 0, "can't create a state type with a nullptr name");
   }
 
-  Type* ret = nullptr;
-
-  ret = new Type (name, name, nullptr, TYPE_STATE, father);
+  Type* ret = new Type(name, name, nullptr, TYPE_STATE, father);
   XBT_DEBUG("StateType %s(%s), child of %s(%s)", ret->name_, ret->id_, father->name_, father->id_);
   LogStateTypeDefinition(ret);
   return ret;