From a11867d76b2e3254878108fe0b83111be8cd2def Mon Sep 17 00:00:00 2001 From: Martin Quinson Date: Sat, 8 Mar 2014 18:47:30 +0100 Subject: [PATCH 1/1] Cosmetics: add some break statement at the end of switches' branches This is to please eclipse source analyser, and yeah this can reduce future bug introduction if another case is added afterward... Well, could be. --- src/surf/platf_generator.c | 3 +++ 1 file changed, 3 insertions(+) diff --git a/src/surf/platf_generator.c b/src/surf/platf_generator.c index 756cf0a849..fadf745abb 100644 --- a/src/surf/platf_generator.c +++ b/src/surf/platf_generator.c @@ -670,6 +670,7 @@ void platf_generate(void) { node_data->router_id = bprintf("router-%d", ++last_router); router_parameters.id = node_data->router_id; sg_platf_new_router(&router_parameters); + break; } } @@ -699,6 +700,7 @@ void platf_generate(void) { break; case HOST: route_parameters.src = src_data->host_parameters.id; + break; } switch(dst_data->kind) { case ROUTER: @@ -709,6 +711,7 @@ void platf_generate(void) { break; case HOST: route_parameters.dst = dst_data->host_parameters.id; + break; } sg_platf_route_begin(&route_parameters); sg_platf_route_add_link(link_parameters->id, &route_parameters); -- 2.20.1