Logo AND Algorithmique Numérique Distribuée

Public GIT Repository
kill an unused variable, fix a segfault after resizing the buffer while varsubsting...
[simgrid.git] / src / xbt / cunit.c
index ab53a53..c22342e 100644 (file)
@@ -16,7 +16,7 @@
 #include "xbt/dynar.h"
 
 /* collection of all suites */
-static xbt_dynar_t _xbt_test_suites = NULL; 
+static xbt_dynar_t _xbt_test_suites = NULL;
 /* global statistics */
 static int _xbt_test_nb_tests = 0;
 static int _xbt_test_test_failed = 0;
@@ -48,14 +48,7 @@ static void xbt_test_log_dump(xbt_test_log_t log) {
   if (log)
     fprintf(stderr,"      log %p(%s:%d)=%s\n",log,log->file,log->line,log->text);
   else
-    fprintf(stderr,"      log=NULL\n");           
-}
-static void xbt_test_log_free(xbt_test_log_t log) {
-  if (!log)
-    return;
-  if (log->text)
-    free(log->text);
-  free(log);
+    fprintf(stderr,"      log=NULL\n");
 }
 
 /* test suite test check */
@@ -74,13 +67,13 @@ static void xbt_test_test_dump(xbt_test_test_t test){
     xbt_test_log_t log;
     unsigned int it_log;
     fprintf(stderr,"    test %p(%s:%d)=%s (%s)\n",
-           test,test->file,test->line,test->title,
-           test->failed?"failed":"not failed");
+            test,test->file,test->line,test->title,
+            test->failed?"failed":"not failed");
     xbt_dynar_foreach(test->logs,it_log,log)
-      xbt_test_log_dump(log);
+    xbt_test_log_dump(log);
   }
   else
-    fprintf(stderr,"    test=NULL\n");    
+    fprintf(stderr,"    test=NULL\n");
 }
 
 /* test suite test unit */
@@ -102,11 +95,11 @@ static void xbt_test_unit_dump(xbt_test_unit_t unit) {
     xbt_test_test_t test;
     unsigned int it_test;
     fprintf(stderr,"  UNIT %s: %s (%s)\n",
-           unit->name,unit->title,
-           (unit->enabled?"enabled":"disabled"));
+            unit->name,unit->title,
+            (unit->enabled?"enabled":"disabled"));
     if (unit->enabled)
       xbt_dynar_foreach(unit->tests,it_test,test)
-       xbt_test_test_dump(test);
+      xbt_test_test_dump(test);
   } else {
     fprintf(stderr,"  unit=NULL\n");
   }
@@ -135,17 +128,37 @@ static void xbt_test_suite_free(void *s) {
   free(suite);
 }
 
+static void xbt_test_unit_free(void *unit) {
+  xbt_test_unit_t u = *(xbt_test_unit_t*)unit;
+  /* name is static */
+  free(u->title);
+  xbt_dynar_free(&u->tests);
+  free(u);
+}
+static void xbt_test_test_free(void *test) {
+  xbt_test_test_t t = *(xbt_test_test_t*)test;
+  free(t->title);
+  xbt_dynar_free(&(t->logs));
+  free(t);
+}
+static void xbt_test_log_free(void *log) {
+  xbt_test_log_t l= *(xbt_test_log_t*) log;
+  free(l->text);
+  free(l);
+}
+
 /** @brief create test suite */
 xbt_test_suite_t xbt_test_suite_new(const char *name, const char *fmt, ...) {
   xbt_test_suite_t suite = xbt_new0(struct s_xbt_test_suite,1);
   va_list ap;
+  int vres;
 
-  if (!_xbt_test_suites) 
+  if (!_xbt_test_suites)
     _xbt_test_suites = xbt_dynar_new(sizeof(xbt_test_suite_t),xbt_test_suite_free);
 
   va_start(ap, fmt);
-  vasprintf(&suite->title,fmt, ap);
-  suite->units = xbt_dynar_new(sizeof(xbt_test_unit_t), NULL);
+  vres = vasprintf(&suite->title,fmt, ap);
+  suite->units = xbt_dynar_new(sizeof(xbt_test_unit_t), &xbt_test_unit_free);
   va_end(ap);
   suite->name = name;
   suite->enabled = 1;
@@ -162,18 +175,19 @@ xbt_test_suite_t xbt_test_suite_by_name(const char *name,const char *fmt, ...) {
 
   char *bufname;
   va_list ap;
+  int vres;
 
   if (_xbt_test_suites)
     xbt_dynar_foreach(_xbt_test_suites, it_suite, suite)
-      if (!strcmp(suite->name,name))
-       return suite;
-  
+    if (!strcmp(suite->name,name))
+      return suite;
+
   va_start(ap, fmt);
-  vasprintf(&bufname,fmt, ap);
+  vres = vasprintf(&bufname,fmt, ap);
   va_end(ap);
   suite = xbt_test_suite_new(name,bufname,NULL);
   free(bufname);
-  
+
   return suite;
 }
 
@@ -182,11 +196,11 @@ void xbt_test_suite_dump(xbt_test_suite_t suite) {
     xbt_test_unit_t unit;
     unsigned int it_unit;
     fprintf(stderr,"TESTSUITE %s: %s (%s)\n",
-           suite->name, suite->title,
-           suite->enabled?"enabled":"disabled");
+            suite->name, suite->title,
+            suite->enabled?"enabled":"disabled");
     if (suite->enabled)
       xbt_dynar_foreach(suite->units,it_unit,unit)
-       xbt_test_unit_dump(unit);
+      xbt_test_unit_dump(unit);
   } else {
     fprintf(stderr,"TESTSUITE IS NULL!\n");
   }
@@ -196,22 +210,23 @@ void xbt_test_suite_dump(xbt_test_suite_t suite) {
 void xbt_test_suite_push(xbt_test_suite_t suite, const char *name, ts_test_cb_t func, const char *fmt, ...) {
   xbt_test_unit_t unit;
   va_list ap;
-  
+  int vres;
+
   xbt_assert(suite);
   xbt_assert(func);
   xbt_assert(fmt);
 
   unit = xbt_new0(struct s_xbt_test_unit,1);
   va_start(ap, fmt);
-  vasprintf(&unit->title, fmt, ap);
+  vres = vasprintf(&unit->title, fmt, ap);
   va_end(ap);
   unit->name = (char*)name;
   unit->func = func;
   unit->file = NULL;
   unit->line = 0;
   unit->enabled = 1;
-  unit->tests = xbt_dynar_new(sizeof(xbt_test_test_t), NULL);
-  
+  unit->tests = xbt_dynar_new(sizeof(xbt_test_test_t), xbt_test_test_free);
+
   xbt_dynar_push(suite->units, &unit);
   return;
 }
@@ -228,6 +243,7 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
   unsigned int it_unit,it_test,it_log;
 
   int first=1; /* for result pretty printing */
+  int vres;
 
   if (suite == NULL)
     return 0;
@@ -239,8 +255,8 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
     int i;
 
     xbt_assert2(suite_len<68,"suite title \"%s\" too long (%d should be less than 68",
-               suite->title,suite_len);
-    
+                suite->title,suite_len);
+
     suite_title[0]=' ';
     for (i=1;i<79;i++)
       suite_title[i]='=';
@@ -249,12 +265,12 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
 
     sprintf(suite_title + 40 - (suite_len+4)/2, "[ %s ]", suite->title);
     suite_title[40 + (suite_len+5)/2] = '=';
-    if (!suite->enabled) 
+    if (!suite->enabled)
       sprintf(suite_title+ 70," DISABLED ");
     fprintf(stderr, "\n%s\n",suite_title);
   }
 
-  if (suite->enabled) {        
+  if (suite->enabled) {
     /* iterate through all tests */
     xbt_dynar_foreach(suite->units, it_unit, unit) {
       /* init unit case counters */
@@ -262,97 +278,97 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
       unit->test_ignore = 0;
       unit->test_failed = 0;
       unit->test_expect = 0;
-      
+
       /* display unit title */
-      asprintf(&cp," Unit: %s ......................................"
-              "......................................", unit->title);
+      vres=asprintf(&cp," Unit: %s ......................................"
+               "......................................", unit->title);
       cp[70] = '\0';
       fprintf(stderr, "%s", cp);
       free(cp);
-      
+
       /* run the test case function */
       _xbt_test_current_unit = unit;
       if (unit->enabled)
-       unit->func();
-      
+        unit->func();
+
       /* iterate through all performed tests to determine status */
       xbt_dynar_foreach(unit->tests,it_test, test) {
-       if (test->ignored) {
-         unit->test_ignore++;
-       } else {
-         unit->nb_tests++;
-         
-         if ( test->failed && !test->expected_failure) unit->test_failed++;
-         if (!test->failed &&  test->expected_failure) unit->test_failed++;
-         if (test->expected_failure)
-           unit->test_expect++;
-       }
+        if (test->ignored) {
+          unit->test_ignore++;
+        } else {
+          unit->nb_tests++;
+
+          if ( test->failed && !test->expected_failure) unit->test_failed++;
+          if (!test->failed &&  test->expected_failure) unit->test_failed++;
+          if (test->expected_failure)
+            unit->test_expect++;
+        }
       }
-    
-    
+
+
       /* Display whether this unit went well */
       if (unit->test_failed > 0 || unit->test_expect) {
-       /* some tests failed (or were supposed to), so do detailed reporting of test case */
-       if (unit->test_failed > 0) {
-         fprintf(stderr, ".. failed\n");
-       } else if (unit->nb_tests) {
-         fprintf(stderr, "...... ok\n"); /* successful, but show about expected */
-       } else {
-         fprintf(stderr, ".... skip\n"); /* shouldn't happen, but I'm a bit lost with this logic */
-       }
-       xbt_dynar_foreach(unit->tests,it_test, test) {
-         file = (test->file != NULL ? test->file : unit->file);
-         line = (test->line != 0    ? test->line : unit->line);
-         fprintf(stderr, "      %s: %s [%s:%d]\n", 
-                 (test->ignored?" SKIP":(test->expected_failure?(test->failed?"EFAIL":"EPASS"):
-                                         (test->failed?" FAIL":" PASS"))),
-                 test->title, file, line);
-         
-         if  ( (test->expected_failure && !test->failed) || (!test->expected_failure && test->failed) ) {
-            xbt_dynar_foreach(test->logs,it_log,log) {
-               file = (log->file != NULL ? log->file : file);
-               line = (log->line != 0    ? log->line : line);
-               fprintf(stderr, "             %s:%d: %s\n", 
-                       file, line,log->text);
-               
-            }
-         }
-       }
-       fprintf(stderr, "    Summary: %d of %d tests failed",unit->test_failed, unit->nb_tests);
-       if (unit->test_ignore) {
-         fprintf(stderr," (%d tests ignored)\n",unit->test_ignore);
-       } else {
-         fprintf(stderr,"\n");
-       }
-       
+        /* some tests failed (or were supposed to), so do detailed reporting of test case */
+        if (unit->test_failed > 0) {
+          fprintf(stderr, ".. failed\n");
+        } else if (unit->nb_tests) {
+          fprintf(stderr, "...... ok\n"); /* successful, but show about expected */
+        } else {
+          fprintf(stderr, ".... skip\n"); /* shouldn't happen, but I'm a bit lost with this logic */
+        }
+        xbt_dynar_foreach(unit->tests,it_test, test) {
+          file = (test->file != NULL ? test->file : unit->file);
+          line = (test->line != 0    ? test->line : unit->line);
+          fprintf(stderr, "      %s: %s [%s:%d]\n",
+                  (test->ignored?" SKIP":(test->expected_failure?(test->failed?"EFAIL":"EPASS"):
+                    (test->failed?" FAIL":" PASS"))),
+                    test->title, file, line);
+
+          if  ( (test->expected_failure && !test->failed) || (!test->expected_failure && test->failed) ) {
+            xbt_dynar_foreach(test->logs,it_log,log) {
+              file = (log->file != NULL ? log->file : file);
+              line = (log->line != 0    ? log->line : line);
+              fprintf(stderr, "             %s:%d: %s\n",
+                      file, line,log->text);
+
+            }
+          }
+        }
+        fprintf(stderr, "    Summary: %d of %d tests failed",unit->test_failed, unit->nb_tests);
+        if (unit->test_ignore) {
+          fprintf(stderr," (%d tests ignored)\n",unit->test_ignore);
+        } else {
+          fprintf(stderr,"\n");
+        }
+
       } else if (!unit->enabled) {
-       fprintf(stderr, " disabled\n"); /* no test were run */
+        fprintf(stderr, " disabled\n"); /* no test were run */
       } else if (unit->nb_tests) {
-       fprintf(stderr, "...... ok\n"); /* successful */
+        fprintf(stderr, "...... ok\n"); /* successful */
       } else  {
-       fprintf(stderr, ".... skip\n"); /* no test were run */
+        fprintf(stderr, ".... skip\n"); /* no test were run */
       }
-      
+
       /* Accumulate test counts into the suite */
       suite->nb_tests      += unit->nb_tests;
       suite->test_failed   += unit->test_failed;
       suite->test_ignore   += unit->test_ignore;
       suite->test_expect   += unit->test_expect;
-      
+
       _xbt_test_nb_tests      += unit->nb_tests;
       _xbt_test_test_failed   += unit->test_failed;
       _xbt_test_test_ignore   += unit->test_ignore;
-      _xbt_test_test_expect   += unit->test_expect;  
-      
+      _xbt_test_test_expect   += unit->test_expect;
+
       /* What's the conclusion of this test anyway? */
       if (unit->nb_tests) {
-       suite->nb_units++;
-       if (unit->test_failed)
-         suite->unit_failed++;
+        suite->nb_units++;
+        if (unit->test_failed)
+          suite->unit_failed++;
       } else if (!unit->enabled) {
-       suite->unit_disabled++;
+        suite->unit_disabled++;
       } else {
-       suite->unit_ignore++;
+        suite->unit_ignore++;
       }
     }
   }
@@ -371,18 +387,18 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
     _xbt_test_suite_ignore++;
   }
 
-  
+
   /* print test suite summary */
   if (suite->enabled) {
 
     fprintf(stderr,
-           " =====================================================================%s\n",
-           (suite->nb_units ? (suite->unit_failed   ? "== FAILED":"====== OK")
-                            : (suite->unit_disabled ? " DISABLED":"==== SKIP")));
-    fprintf(stderr, " Summary: Units: %.0f%% ok (%d units: ", 
-           suite->nb_units?((1-(double)suite->unit_failed/(double)suite->nb_units)*100.0):100.0,
-           suite->nb_units);
-    
+            " =====================================================================%s\n",
+            (suite->nb_units ? (suite->unit_failed   ? "== FAILED":"====== OK")
+                             : (suite->unit_disabled ? " DISABLED":"==== SKIP")));
+    fprintf(stderr, " Summary: Units: %.0f%% ok (%d units: ",
+            suite->nb_units?((1-(double)suite->unit_failed/(double)suite->nb_units)*100.0):100.0,
+                           suite->nb_units);
+
     if (suite->nb_units != suite->unit_failed) {
       fprintf(stderr, "%s%d ok",(first?"":", "),suite->nb_units - suite->unit_failed);
       first = 0;
@@ -400,8 +416,8 @@ static int xbt_test_suite_run(xbt_test_suite_t suite) {
       first = 0;
     }
     fprintf(stderr,")\n          Tests: %.0f%% ok (%d tests: ",
-           suite->nb_tests?((1-(double)suite->test_failed/(double)suite->nb_tests)*100.0):100.0,
-           suite->nb_tests);
+            suite->nb_tests?((1-(double)suite->test_failed/(double)suite->nb_tests)*100.0):100.0,
+                           suite->nb_tests);
 
     first=1;
     if (suite->nb_tests != suite->test_failed) {
@@ -458,7 +474,7 @@ static void apply_selection(char *selection) {
       strcpy(dir, sel);
       done=1;
     }
-    
+
     if (dir[0] == '-') {
       enabling = 0;
       memmove(dir,dir+1,strlen(dir));
@@ -470,7 +486,7 @@ static void apply_selection(char *selection) {
 
     p =strchr(dir,':');
     if (p) {
-      strcpy(unitname,p+1); 
+      strcpy(unitname,p+1);
       strncpy(suitename,dir,p-dir);
       suitename[p-dir]='\0';
     } else {
@@ -483,52 +499,52 @@ static void apply_selection(char *selection) {
     /* Deal with the specific case of 'all' pseudo serie */
     if (!strcmp("all",suitename)) {
       if (unitname[0]!='\0') {
-       fprintf(stderr,"The 'all' pseudo-suite does not accept any unit specification\n");
-       exit(1);
+        fprintf(stderr,"The 'all' pseudo-suite does not accept any unit specification\n");
+        exit(1);
       }
 
       xbt_dynar_foreach(_xbt_test_suites,it_suite,suite) {
-       xbt_dynar_foreach(suite->units,it_unit,unit) {
-         unit->enabled = enabling;
-       }
-       suite->enabled = enabling;
+        xbt_dynar_foreach(suite->units,it_unit,unit) {
+          unit->enabled = enabling;
+        }
+        suite->enabled = enabling;
       }
     } else {
       unsigned int it;
       for (it=0; it< xbt_dynar_length(_xbt_test_suites); it++) {
-       xbt_test_suite_t thissuite=xbt_dynar_get_as(_xbt_test_suites,it,xbt_test_suite_t);
-       if (!strcmp(suitename,thissuite->name)) {
-         /* Do not disable the whole suite when we just want to disable a child */
-         if (enabling || (unitname[0]=='\0'))
-           thissuite->enabled = enabling;
-
-         if (unitname[0]=='\0') {
-           xbt_dynar_foreach(thissuite->units,it_unit,unit){
-             unit->enabled = enabling;
-           }
-         } else { /* act on one child only */
-           unsigned int it2_unit;
-           /* search it, first (we won't reuse it for external loop which gets broken) */
-           for (it2_unit=0; it2_unit< xbt_dynar_length(thissuite->units); it2_unit++) {
-             xbt_test_unit_t thisunit=xbt_dynar_get_as(thissuite->units,it2_unit,xbt_test_unit_t);
-             if (!strcmp(thisunit->name,unitname)) {
-               thisunit->enabled = enabling;
-               break;
-             }
-           } /* search relevant unit */
-           if (it2_unit==xbt_dynar_length(thissuite->units)) {
-             fprintf(stderr,"Suite '%s' has no unit of name '%s'. Cannot apply the selection\n",
-                     suitename,unitname);
-             exit(1);
-           }
-         } /* act on childs (either all or one) */
-
-         break;/* found the relevant serie. We are happy */
-       }
+        xbt_test_suite_t thissuite=xbt_dynar_get_as(_xbt_test_suites,it,xbt_test_suite_t);
+        if (!strcmp(suitename,thissuite->name)) {
+          /* Do not disable the whole suite when we just want to disable a child */
+          if (enabling || (unitname[0]=='\0'))
+            thissuite->enabled = enabling;
+
+          if (unitname[0]=='\0') {
+            xbt_dynar_foreach(thissuite->units,it_unit,unit){
+              unit->enabled = enabling;
+            }
+          } else { /* act on one child only */
+            unsigned int it2_unit;
+            /* search it, first (we won't reuse it for external loop which gets broken) */
+            for (it2_unit=0; it2_unit< xbt_dynar_length(thissuite->units); it2_unit++) {
+              xbt_test_unit_t thisunit=xbt_dynar_get_as(thissuite->units,it2_unit,xbt_test_unit_t);
+              if (!strcmp(thisunit->name,unitname)) {
+                thisunit->enabled = enabling;
+                break;
+              }
+            } /* search relevant unit */
+            if (it2_unit==xbt_dynar_length(thissuite->units)) {
+              fprintf(stderr,"Suite '%s' has no unit of name '%s'. Cannot apply the selection\n",
+                      suitename,unitname);
+              exit(1);
+            }
+          } /* act on childs (either all or one) */
+
+          break;/* found the relevant serie. We are happy */
+        }
       } /* search relevant series */
       if (it==xbt_dynar_length(_xbt_test_suites)) {
-       fprintf(stderr,"No suite of name '%s' found. Cannot apply the selection\n",suitename);
-       exit(1);
+        fprintf(stderr,"No suite of name '%s' found. Cannot apply the selection\n",suitename);
+        exit(1);
       }
     }
 
@@ -542,8 +558,8 @@ void xbt_test_dump(char *selection) {
     unsigned int it_suite;
     xbt_test_suite_t suite;
 
-    xbt_dynar_foreach(_xbt_test_suites,it_suite,suite) 
-      xbt_test_suite_dump(suite);
+    xbt_dynar_foreach(_xbt_test_suites,it_suite,suite)
+    xbt_test_suite_dump(suite);
   } else {
     printf(" No suite defined.");
   }
@@ -556,17 +572,17 @@ int xbt_test_run(char *selection) {
     unsigned int it_suite;
     xbt_test_suite_t suite;
     int first=1;
-    
+
     /* Run all the suites */
-    xbt_dynar_foreach(_xbt_test_suites,it_suite,suite) 
-      xbt_test_suite_run(suite);
+    xbt_dynar_foreach(_xbt_test_suites,it_suite,suite)
+    xbt_test_suite_run(suite);
 
     /* Display some more statistics */
     fprintf(stderr,"\n\n TOTAL: Suites: %.0f%% ok (%d suites: ",
-           _xbt_test_nb_suites
-             ? ((1-(double)_xbt_test_suite_failed/(double)_xbt_test_nb_suites)*100.0)
-             : 100.0,
-           _xbt_test_nb_suites);
+            _xbt_test_nb_suites
+            ? ((1-(double)_xbt_test_suite_failed/(double)_xbt_test_nb_suites)*100.0)
+            : 100.0,
+            _xbt_test_nb_suites);
     if (_xbt_test_nb_suites != _xbt_test_suite_failed) {
       fprintf(stderr, "%d ok",_xbt_test_nb_suites - _xbt_test_suite_failed);
       first = 0;
@@ -575,14 +591,14 @@ int xbt_test_run(char *selection) {
       fprintf(stderr, "%s%d failed",(first?"":", "),_xbt_test_suite_failed);
       first = 0;
     }
-    
+
     if (_xbt_test_suite_ignore) {
       fprintf(stderr, "%s%d ignored",(first?"":", "),_xbt_test_suite_ignore);
       first = 0;
     }
     fprintf(stderr,")\n        Units:  %.0f%% ok (%d units: ",
-           _xbt_test_nb_units?((1-(double)_xbt_test_unit_failed/(double)_xbt_test_nb_units)*100.0):100.0,
-           _xbt_test_nb_units);
+            _xbt_test_nb_units?((1-(double)_xbt_test_unit_failed/(double)_xbt_test_nb_units)*100.0):100.0,
+                              _xbt_test_nb_units);
     first=1;
     if (_xbt_test_nb_units != _xbt_test_unit_failed) {
       fprintf(stderr, "%s%d ok",(first?"":", "),_xbt_test_nb_units - _xbt_test_unit_failed);
@@ -597,8 +613,8 @@ int xbt_test_run(char *selection) {
       first = 0;
     }
     fprintf(stderr,")\n        Tests:  %.0f%% ok (%d tests: ",
-           _xbt_test_nb_tests?((1-(double)_xbt_test_test_failed/(double)_xbt_test_nb_tests)*100.0):100.0,
-           _xbt_test_nb_tests);
+            _xbt_test_nb_tests?((1-(double)_xbt_test_test_failed/(double)_xbt_test_nb_tests)*100.0):100.0,
+                              _xbt_test_nb_tests);
     first=1;
     if (_xbt_test_nb_tests != _xbt_test_test_failed) {
       fprintf(stderr, "%s%d ok",(first?"":", "),_xbt_test_nb_tests - _xbt_test_test_failed);
@@ -615,7 +631,7 @@ int xbt_test_run(char *selection) {
     if (_xbt_test_test_expect) {
       fprintf(stderr, "%s%d expected to fail",(first?"":", "),_xbt_test_test_expect);
     }
-    
+
     fprintf(stderr,")\n");
   } else {
     fprintf(stderr,"No unit to run!\n");
@@ -623,27 +639,30 @@ int xbt_test_run(char *selection) {
   }
   return _xbt_test_unit_failed;
 }
-
+void xbt_test_exit(void) {
+  xbt_dynar_free(&_xbt_test_suites);
+}
 
 /* annotate test case with test */
 void _xbt_test_add(const char*file,int line, const char *fmt, ...) {
   xbt_test_unit_t unit=_xbt_test_current_unit;
   xbt_test_test_t test;
   va_list ap;
-  
+  int vres;
+
   xbt_assert(unit);
   xbt_assert(fmt);
 
-  test = xbt_new(struct s_xbt_test_test,1);
+  test = xbt_new0(struct s_xbt_test_test,1);
   va_start(ap, fmt);
-  vasprintf(&test->title, fmt, ap);
+  vres=vasprintf(&test->title, fmt, ap);
   va_end(ap);
   test->failed = 0;
   test->expected_failure = 0;
   test->ignored = 0;
   test->file = file;
   test->line = line;
-  test->logs = xbt_dynar_new(sizeof(xbt_test_log_t),NULL);
+  test->logs = xbt_dynar_new(sizeof(xbt_test_log_t),xbt_test_log_free);
   xbt_dynar_push(unit->tests,&test);
   return;
 }
@@ -654,17 +673,18 @@ void _xbt_test_fail(const char*file,int line,const char *fmt, ...) {
   xbt_test_test_t test;
   xbt_test_log_t log;
   va_list ap;
-  
+  int vres;
+
   xbt_assert(unit);
   xbt_assert(fmt);
 
   xbt_assert1(xbt_dynar_length(_xbt_test_current_unit->tests),
-             "Test failed even before being declared (broken unit: %s)",
-             unit->title);
+              "Test failed even before being declared (broken unit: %s)",
+              unit->title);
 
   log = xbt_new(struct s_xbt_test_log,1);
   va_start(ap, fmt);
-  vasprintf(&log->text,fmt, ap);
+  vres=vasprintf(&log->text,fmt, ap);
   va_end(ap);
   log->file = file;
   log->line = line;
@@ -682,8 +702,8 @@ void xbt_test_exception(xbt_ex_t e) {
 void xbt_test_expect_failure(void) {
   xbt_test_test_t test;
   xbt_assert1(xbt_dynar_length(_xbt_test_current_unit->tests),
-             "Cannot expect the failure of a test before declaring it (broken unit: %s)",
-             _xbt_test_current_unit->title);
+              "Cannot expect the failure of a test before declaring it (broken unit: %s)",
+              _xbt_test_current_unit->title);
   test = xbt_dynar_getlast_as(_xbt_test_current_unit->tests,xbt_test_test_t);
   test->expected_failure = 1;
 }
@@ -691,11 +711,11 @@ void xbt_test_skip(void) {
   xbt_test_test_t test;
 
   xbt_assert1(xbt_dynar_length(_xbt_test_current_unit->tests),
-             "Test skiped even before being declared (broken unit: %s)",
-             _xbt_test_current_unit->title);
+              "Test skiped even before being declared (broken unit: %s)",
+              _xbt_test_current_unit->title);
 
   test = xbt_dynar_getlast_as(_xbt_test_current_unit->tests,
-                             xbt_test_test_t);
+                              xbt_test_test_t);
   test->ignored = 1;
 }
 
@@ -705,16 +725,17 @@ void _xbt_test_log(const char*file,int line,const char *fmt, ...) {
   xbt_test_test_t test;
   xbt_test_log_t log;
   va_list ap;
+  int vres;
 
   xbt_assert(unit);
   xbt_assert(fmt);
 
   xbt_assert1(xbt_dynar_length(_xbt_test_current_unit->tests),
-             "Test logged into even before being declared (broken test unit: %s)",unit->title);
+              "Test logged into even before being declared (broken test unit: %s)",unit->title);
 
   log = xbt_new(struct s_xbt_test_log,1);
   va_start(ap, fmt);
-  vasprintf(&log->text, fmt, ap);
+  vres=vasprintf(&log->text, fmt, ap);
   va_end(ap);
   log->file = file;
   log->line = line;
@@ -731,13 +752,13 @@ void _xbt_test_log(const char*file,int line,const char *fmt, ...) {
 XBT_TEST_SUITE("cunit","Testsuite mechanism autotest");
 
 XBT_TEST_UNIT("expect",test_expected_failure,"expected failures") {
-    xbt_test_add0("Skipped test");
-    xbt_test_skip(); 
+  xbt_test_add0("Skipped test");
+  xbt_test_skip();
 
-    xbt_test_add2("%s %s","EXPECTED","FAILURE");
-    xbt_test_expect_failure();
-    xbt_test_log2("%s %s","Test","log");
-    xbt_test_fail0("EXPECTED FAILURE");
+  xbt_test_add2("%s %s","EXPECTED","FAILURE");
+  xbt_test_expect_failure();
+  xbt_test_log2("%s %s","Test","log");
+  xbt_test_fail0("EXPECTED FAILURE");
 }
 
 #endif /* SIMGRID_TEST */