please my old friend SonarQube now that it's back

[simgrid.git] / examples / msg / trace-process-migration / trace-process-migration.c

diff --git a/examples/msg/trace-process-migration/trace-process-migration.c b/examples/msg/trace-process-migration/trace-process-migration.c
index 13f3955..59d88a8 100644 (file)
--- a/examples/msg/trace-process-migration/trace-process-migration.c
+++ b/examples/msg/trace-process-migration/trace-process-migration.c
@@ -1,22 +1,11 @@
-/* Copyright (c) 2010-2015. The SimGrid Team.
- * All rights reserved.                                                     */
+/* Copyright (c) 2010-2016. The SimGrid Team. All rights reserved.          */
 
 /* This program is free software; you can redistribute it and/or modify it
  * under the terms of the license (GNU LGPL) which comes with this package. */
 
-/** @addtogroup MSG_examples
- * 
- * - <b>tracing/procmig.c</b> This program shows a process migration. Tracing this program with the options below
- * enables a gantt-chart visualization of where the process has been during its execution. Migrations are represented by
- * arrows from the origin to the destination host. You might want to run this program with the following parameters:
- * --cfg=tracing:yes
- * --cfg=tracing/msg/process:yes
- * (See \ref tracing_tracing_options for details)
- */
-
 #include "simgrid/msg.h"
 
-/** The guy we will move from host to host. It move alone and then is moved by policeman back  */
+/* The guy we will move from host to host. It move alone and then is moved by policeman back  */
 static int emigrant(int argc, char *argv[])
 {
   msg_task_t task = NULL;
@@ -28,7 +17,8 @@ static int emigrant(int argc, char *argv[])
     MSG_task_receive(&(task), "master_mailbox");
     destination = (char*)MSG_task_get_data (task);
     MSG_task_destroy (task);
-    if (!destination) break; //there is no destination, die
+    if (destination == NULL)
+      break; //there is no destination, die
     MSG_process_migrate(MSG_process_self(), MSG_host_by_name(destination));
     MSG_process_sleep(2); // I am tired, have to sleep for 2 seconds
     free (destination);
@@ -39,8 +29,6 @@ static int emigrant(int argc, char *argv[])
 
 static int policeman(int argc, char *argv[])
 {
-  msg_task_t task = NULL;
-
   // I am the master of emigrant process,
   // I tell it where it must emigrate to.
   xbt_dynar_t destinations = xbt_dynar_new (sizeof(char*), &xbt_free_ref);
@@ -57,13 +45,12 @@ static int policeman(int argc, char *argv[])
   char *destination;
   unsigned int i;
   xbt_dynar_foreach(destinations, i, destination){
-    task = MSG_task_create("task", 0, 0, NULL);
-    if (destination){
+    msg_task_t task = MSG_task_create("task", 0, 0, NULL);
+    if (destination != NULL){
       MSG_task_set_data(task, xbt_strdup (destination));
     }
     MSG_task_set_category(task, "migration_order");
     MSG_task_send (task, "master_mailbox");
-    task = NULL;
   }
   xbt_dynar_free (&destinations);
   return 0;