X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/9e2f30eca15bf2c12a75bf0d59ee82f7c4553193..51d4edd27c6eb85f2db6df6d531d1538739cfbb3:/src/smpi/smpi_group.cpp

diff --git a/src/smpi/smpi_group.cpp b/src/smpi/smpi_group.cpp
index ea72ca5d0e..41eb2456a1 100644
--- a/src/smpi/smpi_group.cpp
+++ b/src/smpi/smpi_group.cpp
@@ -17,8 +17,8 @@ typedef struct s_smpi_mpi_group {
 
 static s_smpi_mpi_group_t mpi_MPI_GROUP_EMPTY = {
   0,                            /* size */
-  NULL,                         /* rank_to_index_map */
-  NULL,                         /* index_to_rank_map */
+  nullptr,                         /* rank_to_index_map */
+  nullptr,                         /* index_to_rank_map */
   1,                            /* refcount: start > 0 so that this group never gets freed */
 };
 
@@ -46,7 +46,7 @@ MPI_Group smpi_group_copy(MPI_Group origin)
   MPI_Group group=origin;
   char *key;
   char *ptr_rank;
-  xbt_dict_cursor_t cursor = NULL;
+  xbt_dict_cursor_t cursor = nullptr;
   
   int i;
   if(origin != MPI_GROUP_NULL
@@ -62,9 +62,9 @@ MPI_Group smpi_group_copy(MPI_Group origin)
       }
 
       xbt_dict_foreach(origin->index_to_rank_map, cursor, key, ptr_rank) {
-        int * cp = (int*)xbt_malloc(sizeof(int));
-        *cp=*(int*)ptr_rank;
-        xbt_dict_set(group->index_to_rank_map, key, cp, NULL);
+        int * cp = static_cast<int*>(xbt_malloc(sizeof(int)));
+        *cp=*reinterpret_cast<int*>(ptr_rank);
+        xbt_dict_set(group->index_to_rank_map, key, cp, nullptr);
       }
     }
 
@@ -86,12 +86,12 @@ void smpi_group_set_mapping(MPI_Group group, int index, int rank)
   if (rank < group->size) {
     group->rank_to_index_map[rank] = index;
     if (index!=MPI_UNDEFINED ) {
-      val_rank = (int *) malloc(sizeof(int));
+      val_rank = static_cast<int *>(xbt_malloc(sizeof(int)));
       *val_rank = rank;
 
       char * key = bprintf("%d", index);
-      xbt_dict_set(group->index_to_rank_map, key, val_rank, NULL);
-      free(key);
+      xbt_dict_set(group->index_to_rank_map, key, val_rank, nullptr);
+      xbt_free(key);
     }
   }
 }
@@ -108,12 +108,14 @@ int smpi_group_index(MPI_Group group, int rank)
 
 int smpi_group_rank(MPI_Group group, int index)
 {
-  int * ptr_rank = NULL;
+  int * ptr_rank = nullptr;
+  if (group==MPI_GROUP_EMPTY)
+    return MPI_UNDEFINED;
   char * key = bprintf("%d", index);
   ptr_rank = static_cast<int*>(xbt_dict_get_or_null(group->index_to_rank_map, key));
   xbt_free(key);
 
-  if (!ptr_rank)
+  if (ptr_rank==nullptr)
     return MPI_UNDEFINED;
   return *ptr_rank;
 }