From: Martin Quinson <martin.quinson@loria.fr>
Date: Sun, 7 Feb 2016 12:47:33 +0000 (+0100)
Subject: rename a struct to obey the C++ One Definition Rule (and please gcc6)
X-Git-Tag: v3_13~954
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/92882866aab7e49d993fc3965bc1010146fcac35

rename a struct to obey the C++ One Definition Rule (and please gcc6)
---

diff --git a/src/smpi/smpi_base.cpp b/src/smpi/smpi_base.cpp
index ddaffbc077..91d56273c7 100644
--- a/src/smpi/smpi_base.cpp
+++ b/src/smpi/smpi_base.cpp
@@ -62,12 +62,12 @@ static int match_send(void* a, void* b,smx_synchro_t ignored) {
 }
 
 
-typedef struct s_smpi_factor *smpi_factor_t;
-typedef struct s_smpi_factor {
+typedef struct s_smpi_os_factor *smpi_os_factor_t;
+typedef struct s_smpi_os_factor {
   long factor;
   int nb_values;
   double values[4];//arbitrary set to 4
-} s_smpi_factor_t;
+} s_smpi_os_factor_t;
 xbt_dynar_t smpi_os_values = NULL;
 xbt_dynar_t smpi_or_values = NULL;
 xbt_dynar_t smpi_ois_values = NULL;
@@ -79,12 +79,12 @@ double smpi_test_sleep = 1e-4;
 
 // Methods used to parse and store the values for timing injections in smpi
 // These are taken from surf/network.c and generalized to have more factors
-// These methods should be merged with those in surf/network.c (moved somewhere in xbt ?)
+// FIXME: These methods should be merged with those in surf/network.c (moved somewhere in xbt ?)
 
 static int factor_cmp(const void *pa, const void *pb)
 {
-  return (((s_smpi_factor_t*)pa)->factor > ((s_smpi_factor_t*)pb)->factor) ? 1 :
-         (((s_smpi_factor_t*)pa)->factor < ((s_smpi_factor_t*)pb)->factor) ? -1 : 0;
+  return (((s_smpi_os_factor_t*)pa)->factor > ((s_smpi_os_factor_t*)pb)->factor) ? 1 :
+         (((s_smpi_os_factor_t*)pa)->factor < ((s_smpi_os_factor_t*)pb)->factor) ? -1 : 0;
 }
 
 
@@ -92,15 +92,15 @@ static xbt_dynar_t parse_factor(const char *smpi_coef_string)
 {
   char *value = NULL;
   unsigned int iter = 0;
-  s_smpi_factor_t fact;
+  s_smpi_os_factor_t fact;
   fact.nb_values=0;
   unsigned int i=0;
   xbt_dynar_t smpi_factor, radical_elements, radical_elements2 = NULL;
 
-  smpi_factor = xbt_dynar_new(sizeof(s_smpi_factor_t), NULL);
+  smpi_factor = xbt_dynar_new(sizeof(s_smpi_os_factor_t), NULL);
   radical_elements = xbt_str_split(smpi_coef_string, ";");
   xbt_dynar_foreach(radical_elements, iter, value) {
-    memset(&fact, 0, sizeof(s_smpi_factor_t));
+    memset(&fact, 0, sizeof(s_smpi_os_factor_t));
     radical_elements2 = xbt_str_split(value, ":");
     if (xbt_dynar_length(radical_elements2) <2 || xbt_dynar_length(radical_elements2) > 5)
       xbt_die("Malformed radical for smpi factor!");
@@ -113,7 +113,7 @@ static xbt_dynar_t parse_factor(const char *smpi_coef_string)
         }
     }
 
-    xbt_dynar_push_as(smpi_factor, s_smpi_factor_t, fact);
+    xbt_dynar_push_as(smpi_factor, s_smpi_os_factor_t, fact);
     XBT_DEBUG("smpi_factor:\t%ld : %d values, first: %f", fact.factor, fact.nb_values ,fact.values[0]);
     xbt_dynar_free(&radical_elements2);
   }
@@ -133,7 +133,7 @@ static double smpi_os(double size)
     smpi_register_static(smpi_os_values, xbt_dynar_free_voidp);
   }
   unsigned int iter = 0;
-  s_smpi_factor_t fact;
+  s_smpi_os_factor_t fact;
   double current=0.0;
   // Iterate over all the sections that were specified and find the right
   // value. (fact.factor represents the interval sizes; we want to find the
@@ -162,7 +162,7 @@ static double smpi_ois(double size)
     smpi_register_static(smpi_ois_values, xbt_dynar_free_voidp);
   }
   unsigned int iter = 0;
-  s_smpi_factor_t fact;
+  s_smpi_os_factor_t fact;
   double current=0.0;
   // Iterate over all the sections that were specified and find the right
   // value. (fact.factor represents the interval sizes; we want to find the
@@ -191,7 +191,7 @@ static double smpi_or(double size)
     smpi_register_static(smpi_or_values, xbt_dynar_free_voidp);
   }
   unsigned int iter = 0;
-  s_smpi_factor_t fact;
+  s_smpi_os_factor_t fact;
   double current=0.0;
   // Iterate over all the sections that were specified and find the right
   // value. (fact.factor represents the interval sizes; we want to find the