From: Lucas Schnorr <Lucas.Schnorr@imag.fr>
Date: Thu, 14 Feb 2013 13:13:56 +0000 (+0100)
Subject: [trace] remove useless if, fix identation
X-Git-Tag: v3_9_90~491^2~30^2~1
X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/0d40830c5313be63de41401578c6eda0725c85ed?ds=sidebyside

[trace] remove useless if, fix identation
---

diff --git a/src/surf/surf_routing_generic.c b/src/surf/surf_routing_generic.c
index 94fbd5fbf1..ea3726452b 100644
--- a/src/surf/surf_routing_generic.c
+++ b/src/surf/surf_routing_generic.c
@@ -160,41 +160,40 @@ void generic_get_graph(xbt_graph_t graph, xbt_dict_t nodes, xbt_dict_t edges,
 
       XBT_DEBUG ("get_route_and_latency %s -> %s", my_src->name, my_dst->name);
 
-      if (route) {
-        unsigned int cpt;
-        void *link;
-
-        xbt_node_t current, previous;
-        const char *previous_name, *current_name;
-
-        if (route->gw_src) {
-          previous = new_xbt_graph_node(graph, route->gw_src->name, nodes);
-          previous_name = route->gw_src->name;
-        } else {
-          previous = new_xbt_graph_node(graph, my_src->name, nodes);
-          previous_name = my_src->name;
-        }
-
-        xbt_dynar_foreach(route->link_list, cpt, link) {
-          char *link_name = ((surf_resource_t) link)->name;
-          current = new_xbt_graph_node(graph, link_name, nodes);
-          current_name = link_name;
-          new_xbt_graph_edge(graph, previous, current, edges);
-          XBT_DEBUG ("  %s -> %s", previous_name, current_name);
-          previous = current;
-          previous_name = current_name;
-        }
+      unsigned int cpt;
+      void *link;
+
+      xbt_node_t current, previous;
+      const char *previous_name, *current_name;
+
+      if (route->gw_src) {
+        previous = new_xbt_graph_node(graph, route->gw_src->name, nodes);
+        previous_name = route->gw_src->name;
+      } else {
+        previous = new_xbt_graph_node(graph, my_src->name, nodes);
+        previous_name = my_src->name;
+      }
 
-        if (route->gw_dst) {
-          current = new_xbt_graph_node(graph, route->gw_dst->name, nodes);
-          current_name = route->gw_dst->name;
-        } else {
-          current = new_xbt_graph_node(graph, my_dst->name, nodes);
-          current_name = my_dst->name;
-        }
+      xbt_dynar_foreach(route->link_list, cpt, link) {
+        char *link_name = ((surf_resource_t) link)->name;
+        current = new_xbt_graph_node(graph, link_name, nodes);
+        current_name = link_name;
         new_xbt_graph_edge(graph, previous, current, edges);
         XBT_DEBUG ("  %s -> %s", previous_name, current_name);
+        previous = current;
+        previous_name = current_name;
+      }
+
+      if (route->gw_dst) {
+        current = new_xbt_graph_node(graph, route->gw_dst->name, nodes);
+        current_name = route->gw_dst->name;
+      } else {
+        current = new_xbt_graph_node(graph, my_dst->name, nodes);
+        current_name = my_dst->name;
       }
+      new_xbt_graph_edge(graph, previous, current, edges);
+      XBT_DEBUG ("  %s -> %s", previous_name, current_name);
+
       xbt_dynar_free (&(route->link_list));
       xbt_free (route);
     }