From: Martin Quinson Date: Sat, 8 Mar 2014 17:47:30 +0000 (+0100) Subject: Cosmetics: add some break statement at the end of switches' branches X-Git-Tag: v3_11~236 X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/commitdiff_plain/a11867d76b2e3254878108fe0b83111be8cd2def?ds=sidebyside;hp=abcdd6162e7e02326763aaad812b9e60a2bade68 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. --- 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);