SD_task_t simplification

[simgrid.git] / src / instr / jedule / jedule_output.cpp

diff --git a/src/instr/jedule/jedule_output.cpp b/src/instr/jedule/jedule_output.cpp
index 6c0ceb4..2b28d09 100644 (file)
--- a/src/instr/jedule/jedule_output.cpp
+++ b/src/instr/jedule/jedule_output.cpp
@@ -24,11 +24,11 @@ xbt_dynar_t jedule_event_list;
 static FILE *jed_file;
 
 static void get_hierarchy_list(xbt_dynar_t hier_list, jed_simgrid_container_t container) {
-  xbt_assert( container != NULL );
+  xbt_assert( container != nullptr );
 
-  if( container->parent != NULL ) {
+  if( container->parent != nullptr ) {
 
-    if( container->parent->container_children == NULL ) {
+    if( container->parent->container_children == nullptr ) {
       // we are in the last level
       get_hierarchy_list(hier_list, container->parent);
     } else {
@@ -54,7 +54,7 @@ static void get_hierarchy_list(xbt_dynar_t hier_list, jed_simgrid_container_t co
   }
 }
 
-static void get_hierarchy_string(jed_simgrid_container_t container, char *outbuf) {
+static void get_hierarchy_string(jed_simgrid_container_t container, char *outbuf, int bufsize) {
     char buf[STR_BUF_SIZE];
     xbt_dynar_t hier_list;
     unsigned int iter;
@@ -62,7 +62,7 @@ static void get_hierarchy_string(jed_simgrid_container_t container, char *outbuf
     unsigned int length;
 
     outbuf[0] = '\0';
-    hier_list = xbt_dynar_new(sizeof(int), NULL);
+    hier_list = xbt_dynar_new(sizeof(int), nullptr);
     get_hierarchy_list(hier_list, container);
 
     length = xbt_dynar_length(hier_list);
@@ -73,17 +73,17 @@ static void get_hierarchy_string(jed_simgrid_container_t container, char *outbuf
         } else {
             snprintf(buf, STR_BUF_SIZE, "%d", number);
         }
-        strncat(outbuf, buf, STR_BUF_SIZE-strlen(outbuf));
+        strncat(outbuf, buf, bufsize-strlen(outbuf));
     }
 
     xbt_dynar_free(&hier_list);
 }
 
 static void print_key_value_dict(xbt_dict_t key_value_dict) {
-  xbt_dict_cursor_t cursor=NULL;
+  xbt_dict_cursor_t cursor=nullptr;
   char *key,*data;
 
-  if( key_value_dict != NULL ) {
+  if( key_value_dict != nullptr ) {
     xbt_dict_foreach(key_value_dict,cursor,key,data) {
       fprintf(jed_file, "        <prop key=\"%s\" value=\"%s\" />\n",key,data);
     }
@@ -94,12 +94,12 @@ static void print_resources(jed_simgrid_container_t resource_parent) {
   unsigned int res_nb;
   unsigned int i;
   char *res_name;
-    char resid[STR_BUF_SIZE];
-  xbt_assert( resource_parent->resource_list != NULL );
+  char resid[STR_BUF_SIZE];
+  xbt_assert( resource_parent->resource_list != nullptr );
 
   res_nb = xbt_dynar_length(resource_parent->resource_list);
 
-  get_hierarchy_string(resource_parent, resid);
+  get_hierarchy_string(resource_parent, resid, STR_BUF_SIZE);
 
   fprintf(jed_file, "      <rset id=\"%s\" nb=\"%d\" names=\"", resid, res_nb);
   xbt_dynar_foreach(resource_parent->resource_list, i, res_name) {
@@ -115,10 +115,10 @@ static void print_container(jed_simgrid_container_t container) {
   unsigned int i;
   jed_simgrid_container_t child_container;
 
-  xbt_assert( container != NULL );
+  xbt_assert( container != nullptr );
 
   fprintf(jed_file, "    <res name=\"%s\">\n", container->name);
-  if( container->container_children != NULL ) {
+  if( container->container_children != nullptr ) {
     xbt_dynar_foreach(container->container_children, i, child_container) {
       print_container(child_container);
     }
@@ -138,8 +138,8 @@ static void print_event(jed_event_t event) {
   unsigned int i;
   jed_res_subset_t subset;
 
-  xbt_assert( event != NULL );
-  xbt_assert( event->resource_subsets != NULL );
+  xbt_assert( event != nullptr );
+  xbt_assert( event->resource_subsets != nullptr );
 
   fprintf(jed_file, "    <event>\n");
 
@@ -155,7 +155,7 @@ static void print_event(jed_event_t event) {
     int end   = subset->start_idx + subset->nres - 1;
     char resid[STR_BUF_SIZE];
 
-    get_hierarchy_string(subset->parent, resid);
+    get_hierarchy_string(subset->parent, resid, STR_BUF_SIZE);
     fprintf(jed_file, "        <select resources=\"");
     fprintf(jed_file, "%s", resid);
     fprintf(jed_file, ".[%d-%d]", start, end);
@@ -229,7 +229,7 @@ void jedule_cleanup_output() {
 }
 
 void jedule_store_event(jed_event_t event) {
-  xbt_assert(event != NULL);
+  xbt_assert(event != nullptr);
   xbt_dynar_push(jedule_event_list, &event);
 }
 #endif