From 40dffe1f6c370e36f3e20ad82539e69b25cdcecd Mon Sep 17 00:00:00 2001 From: navarro Date: Mon, 19 Mar 2012 13:06:53 +0100 Subject: [PATCH 1/1] Use for push_as and get_as for dynar --- src/surf/surf_routing_cluster.c | 6 ++++-- src/surf/surf_routing_full.c | 12 +++++++----- src/surf/surf_routing_generic.c | 4 ++-- src/surf/surf_routing_none.c | 6 ++++-- 4 files changed, 17 insertions(+), 11 deletions(-) diff --git a/src/surf/surf_routing_cluster.c b/src/surf/surf_routing_cluster.c index a34dc31c33..9a8a4115b6 100644 --- a/src/surf/surf_routing_cluster.c +++ b/src/surf/surf_routing_cluster.c @@ -59,12 +59,14 @@ static void model_cluster_finalize(AS_t as) { } static int cluster_parse_PU(AS_t rc, network_element_t elm) { - xbt_dynar_push(rc->index_network_elm, (void *)elm); + XBT_DEBUG("Load process unit \"%s\"", elm->name); + xbt_dynar_push_as(rc->index_network_elm,network_element_t,elm); return xbt_dynar_length(rc->index_network_elm)-1; } static int cluster_parse_AS(AS_t rc, network_element_t elm) { - xbt_dynar_push(rc->index_network_elm, (void *)elm); + XBT_DEBUG("Load Autonomous system \"%s\"", elm->name); + xbt_dynar_push_as(rc->index_network_elm,network_element_t,elm); return xbt_dynar_length(rc->index_network_elm)-1; } diff --git a/src/surf/surf_routing_full.c b/src/surf/surf_routing_full.c index 1ac2f26306..bafd2faeeb 100644 --- a/src/surf/surf_routing_full.c +++ b/src/surf/surf_routing_full.c @@ -38,17 +38,19 @@ static xbt_dynar_t full_get_onelink_routes(AS_t rc) void *link = *(void **) xbt_dynar_get_ptr(route->link_list, 0); onelink_t onelink = xbt_new0(s_onelink_t, 1); onelink->link_ptr = link; - if (routing->generic_routing.hierarchy == SURF_ROUTING_BASE) { - onelink->src = xbt_dynar_get_as(routing->generic_routing.index_network_elm,src,network_element_t); + if (rc->hierarchy == SURF_ROUTING_BASE) { + onelink->src = xbt_dynar_get_as(rc->index_network_elm,src,network_element_t); onelink->src->id = src; - onelink->dst = xbt_dynar_get_as(routing->generic_routing.index_network_elm,dst,network_element_t); + onelink->dst = xbt_dynar_get_as(rc->index_network_elm,dst,network_element_t); onelink->dst->id = dst; - } else if (routing->generic_routing.hierarchy == - SURF_ROUTING_RECURSIVE) { + } else if (rc->hierarchy == SURF_ROUTING_RECURSIVE) { onelink->src = route->src_gateway; onelink->dst = route->dst_gateway; } xbt_dynar_push(ret, &onelink); + XBT_DEBUG("Push route from '%d' to '%d'", + src, + dst); } } } diff --git a/src/surf/surf_routing_generic.c b/src/surf/surf_routing_generic.c index 4301a34250..38cc16d09e 100644 --- a/src/surf/surf_routing_generic.c +++ b/src/surf/surf_routing_generic.c @@ -41,14 +41,14 @@ void model_generic_finalize(AS_t as) { int generic_parse_PU(AS_t as, network_element_t elm) { XBT_DEBUG("Load process unit \"%s\"", elm->name); - xbt_dynar_push(as->index_network_elm, elm); + xbt_dynar_push_as(as->index_network_elm,network_element_t,elm); return xbt_dynar_length(as->index_network_elm)-1; } int generic_parse_AS(AS_t as, network_element_t elm) { XBT_DEBUG("Load Autonomous system \"%s\"", elm->name); - xbt_dynar_push(as->index_network_elm, elm); + xbt_dynar_push_as(as->index_network_elm,network_element_t,elm); return xbt_dynar_length(as->index_network_elm)-1; } diff --git a/src/surf/surf_routing_none.c b/src/surf/surf_routing_none.c index 726a873ba2..d4a598df46 100644 --- a/src/surf/surf_routing_none.c +++ b/src/surf/surf_routing_none.c @@ -24,13 +24,15 @@ static route_t none_get_bypass_route(AS_t rc, } static int none_parse_PU(AS_t rc, network_element_t elm) { - xbt_dynar_push(rc->index_network_elm, (void *)elm); + XBT_DEBUG("Load process unit \"%s\"", elm->name); + xbt_dynar_push_as(rc->index_network_elm,network_element_t,elm); /* don't care about PUs */ return -1; } static int none_parse_AS(AS_t rc, network_element_t elm) { - xbt_dynar_push(rc->index_network_elm, (void *)elm); + XBT_DEBUG("Load Autonomous system \"%s\"", elm->name); + xbt_dynar_push_as(rc->index_network_elm,network_element_t,elm); /* even don't care about sub-ASes -- I'm as nihilist as an old punk*/ return -1; } -- 2.20.1