X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/4e0b0a957106c8c55b7aa24f118ed5862e98064d..35471ecdf533b2701ca99d3921c42944531883ce:/src/xbt/config.c diff --git a/src/xbt/config.c b/src/xbt/config.c index 0b6474bc07..a6c8f6e59a 100644 --- a/src/xbt/config.c +++ b/src/xbt/config.c @@ -1,19 +1,18 @@ /* $Id$ */ -/* config - Dictionnary where the type of each cell is provided. */ +/* config - Dictionnary where the type of each variable is provided. */ /* This is useful to build named structs, like option or property sets. */ -/* Authors: Martin Quinson */ -/* Copyright (C) 2001,2002,2003,2004 the OURAGAN project. */ +/* Copyright (c) 2001,2002,2003,2004 Martin Quinson. 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. */ + * under the terms of the license (GNU LGPL) which comes with this package. */ #include "xbt/misc.h" #include "xbt/sysdep.h" #include "xbt/log.h" -#include "xbt/error.h" +#include "xbt/ex.h" #include "xbt/dynar.h" #include "xbt/dict.h" @@ -21,15 +20,19 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(config,xbt,"configuration support"); -/* xbt_cfgelm_t: the typedef corresponding to a config cell. +/* xbt_cfgelm_t: the typedef corresponding to a config variable. Both data and DTD are mixed, but fixing it now would prevent me to ever defend my thesis. */ typedef struct { - /* Allowed type of the cell */ + /* Allowed type of the variable */ e_xbt_cfgelm_type_t type; int min,max; + + /* Callbacks */ + xbt_cfg_cb_t cb_set; + xbt_cfg_cb_t cb_rm; /* actual content (cannot be an union because type host uses both str and i) */ @@ -37,36 +40,29 @@ typedef struct { } s_xbt_cfgelm_t,*xbt_cfgelm_t; static const char *xbt_cfgelm_type_name[xbt_cfgelm_type_count]= - {"int","double","string","host"}; + {"int","double","string","host","any"}; -/* Internal stuff used in cache to free a cell */ +/* Internal stuff used in cache to free a variable */ static void xbt_cfgelm_free(void *data); -/* Retrieve the cell we'll modify */ -static xbt_error_t xbt_cfgelm_get(xbt_cfg_t cfg, const char *name, - e_xbt_cfgelm_type_t type, - /* OUT */ xbt_cfgelm_t *whereto); - -void xbt_cfg_str_free(void *d); -void xbt_cfg_host_free(void *d); +/* Retrieve the variable we'll modify */ +static xbt_cfgelm_t xbt_cfgelm_get(xbt_cfg_t cfg, const char *name, + e_xbt_cfgelm_type_t type); -void xbt_cfg_str_free(void *d){ - xbt_free(*(void**)d); +static void xbt_cfg_str_free(void *d){ + free(*(void**)d); } -void xbt_cfg_host_free(void *d){ +static void xbt_cfg_host_free(void *d){ xbt_host_t *h=(xbt_host_t*) *(void**)d; if (h) { - if (h->name) xbt_free(h->name); - xbt_free(h); + if (h->name) free(h->name); + free(h); } } /*----[ Memory management ]-----------------------------------------------*/ -/** - * xbt_cfg_new: - * - * @whereto: +/** @brief Constructor * * Initialise an config set */ @@ -76,45 +72,44 @@ xbt_cfg_t xbt_cfg_new(void) { return (xbt_cfg_t)xbt_dict_new(); } -/** - * xbt_cfg_cpy: +/** \brief Copy an existing configuration set * - * @whereto: the config set to be created - * @tocopy: the source data + * \arg whereto the config set to be created + * \arg tocopy the source data * + * This only copy the registrations, not the actual content */ void xbt_cfg_cpy(xbt_cfg_t tocopy,xbt_cfg_t *whereto) { xbt_dict_cursor_t cursor=NULL; - xbt_cfgelm_t cell=NULL; + xbt_cfgelm_t variable=NULL; char *name=NULL; *whereto=NULL; xbt_assert0(tocopy,"cannot copy NULL config"); - xbt_dict_foreach((xbt_dict_t)tocopy,cursor,name,cell) { - xbt_cfg_register(*whereto, name, cell->type, cell->min, cell->max); + xbt_dict_foreach((xbt_dict_t)tocopy,cursor,name,variable) { + xbt_cfg_register(*whereto, name, variable->type, variable->min, variable->max, + variable->cb_set, variable->cb_rm); } } +/** @brief Destructor */ void xbt_cfg_free(xbt_cfg_t *cfg) { xbt_dict_free((xbt_dict_t*)cfg); } -/** - * xbt_cfg_dump: - * @name: The name to give to this config set - * @indent: what to write at the begining of each line (right number of spaces) - * @cfg: the config set +/** @brief Dump a config set for debuging purpose * - * Dumps a config set for debuging purpose + * \arg name The name to give to this config set + * \arg indent what to write at the begining of each line (right number of spaces) + * \arg cfg the config set */ -void -xbt_cfg_dump(const char *name,const char *indent,xbt_cfg_t cfg) { +void xbt_cfg_dump(const char *name,const char *indent,xbt_cfg_t cfg) { xbt_dict_t dict = (xbt_dict_t) cfg; xbt_dict_cursor_t cursor=NULL; - xbt_cfgelm_t cell=NULL; + xbt_cfgelm_t variable=NULL; char *key=NULL; int i; int size; @@ -125,41 +120,42 @@ xbt_cfg_dump(const char *name,const char *indent,xbt_cfg_t cfg) { if (name) printf("%s>> Dumping of the config set '%s':\n",indent,name); - xbt_dict_foreach(dict,cursor,key,cell) { + xbt_dict_foreach(dict,cursor,key,variable) { printf("%s %s:",indent,key); - size = xbt_dynar_length(cell->content); - printf("%d_to_%d_%s. Actual size=%d. List of values:\n", - cell->min,cell->max,xbt_cfgelm_type_name[cell->type], - size); + size = xbt_dynar_length(variable->content); + printf("%d_to_%d_%s. Actual size=%d. prerm=%p,postset=%p, List of values:\n", + variable->min,variable->max,xbt_cfgelm_type_name[variable->type], + size, + variable->cb_rm, variable->cb_set); - switch (cell->type) { + switch (variable->type) { case xbt_cfgelm_int: for (i=0; icontent,i,int); + ival = xbt_dynar_get_as(variable->content,i,int); printf ("%s %d\n",indent,ival); } break; case xbt_cfgelm_double: for (i=0; icontent,i,double); + dval = xbt_dynar_get_as(variable->content,i,double); printf ("%s %f\n",indent,dval); } break; case xbt_cfgelm_string: for (i=0; icontent,i,char*); + sval = xbt_dynar_get_as(variable->content,i,char*); printf ("%s %s\n",indent,sval); } break; case xbt_cfgelm_host: for (i=0; icontent,i,xbt_host_t*); + hval = xbt_dynar_get_as(variable->content,i,xbt_host_t*); printf ("%s %s:%d\n",indent,hval->name,hval->port); } break; @@ -177,11 +173,7 @@ xbt_cfg_dump(const char *name,const char *indent,xbt_cfg_t cfg) { return; } -/** - * xbt_cfgelm_free: - * - * @data: the data to be freed (typed as void* to be usable as free funct in dict) - * +/* * free an config element */ @@ -190,42 +182,57 @@ void xbt_cfgelm_free(void *data) { if (!c) return; xbt_dynar_free(&(c->content)); - xbt_free(c); + free(c); } /*----[ Registering stuff ]-----------------------------------------------*/ -/** - * xbt_cfg_register: - * @cfg: the config set - * @type: the type of the config element - * @min: the minimum - * @max: the maximum +/** @brief Register an element within a config set * - * register an element within a config set + * @arg cfg the config set + * @arg type the type of the config element + * @arg min the minimum + * @arg max the maximum */ void xbt_cfg_register(xbt_cfg_t cfg, - const char *name, e_xbt_cfgelm_type_t type, - int min, int max){ + const char *name, e_xbt_cfgelm_type_t type, + int min, int max, + xbt_cfg_cb_t cb_set, xbt_cfg_cb_t cb_rm){ xbt_cfgelm_t res; - xbt_error_t errcode; - - DEBUG4("Register cfg elm %s (%d to %d %s)",name,min,max,xbt_cfgelm_type_name[type]); - errcode = xbt_dict_get((xbt_dict_t)cfg,name,(void**)&res); + xbt_ex_t e; + int found=0; + + xbt_assert(cfg); + xbt_assert4(type>=xbt_cfgelm_int && type<=xbt_cfgelm_host, + "type of %s not valid (%d should be between %d and %d)", + name,type,xbt_cfgelm_int, xbt_cfgelm_host); + DEBUG5("Register cfg elm %s (%d to %d %s (=%d))", + name,min,max,xbt_cfgelm_type_name[type],type); + TRY { + res = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + if (e.category == not_found_error) { + found = 1; + xbt_ex_free(e); + } else { + RETHROW; + } + } - if (errcode == no_error) { + if (!found) { WARN1("Config elem %s registered twice.",name); /* Will be removed by the insertion of the new one */ } - xbt_assert_error(mismatch_error); res=xbt_new(s_xbt_cfgelm_t,1); res->type=type; res->min=min; res->max=max; + res->cb_set = cb_set; + res->cb_rm = cb_rm; switch (type) { case xbt_cfgelm_int: @@ -251,31 +258,32 @@ xbt_cfg_register(xbt_cfg_t cfg, xbt_dict_set((xbt_dict_t)cfg,name,res,&xbt_cfgelm_free); } -/** - * xbt_cfg_unregister: +/** @brief Unregister an element from a config set. * - * @cfg: the config set - * @name: the name of the elem to be freed + * @arg cfg the config set + * @arg name the name of the elem to be freed * - * unregister an element from a config set. - * Note that it removes both the DTD and the actual content. + * Note that it removes both the description and the actual content. + * Throws not_found when no such element exists. */ -xbt_error_t +void xbt_cfg_unregister(xbt_cfg_t cfg,const char *name) { - return xbt_dict_remove((xbt_dict_t)cfg,name); + xbt_dict_remove((xbt_dict_t)cfg,name); } /** - * xbt_cfg_register_str: + * @brief Parse a string and register the stuff described. * - * @cfg: the config set - * @entry: a string describing the element to register + * @arg cfg the config set + * @arg entry a string describing the element to register * - * Parse a string and register the stuff described. + * The string may consist in several variable descriptions separated by a space. + * Each of them must use the following syntax: \:\_to_\_\ + * with type being one of 'string','int', 'host' or 'double'. */ -xbt_error_t +void xbt_cfg_register_str(xbt_cfg_t cfg,const char *entry) { char *entrycpy=xbt_strdup(entry); char *tok; @@ -284,263 +292,231 @@ xbt_cfg_register_str(xbt_cfg_t cfg,const char *entry) { e_xbt_cfgelm_type_t type; tok=strchr(entrycpy, ':'); - if (!tok) { - ERROR3("%s%s%s", - "Invalid config element descriptor: ",entry, - "; Should be :_to__"); - xbt_free(entrycpy); - xbt_abort(); - } + xbt_assert2(tok,"Invalid config element descriptor: %s%s", + entry, + "; Should be :_to__"); *(tok++)='\0'; min=strtol(tok, &tok, 10); - if (!tok) { - ERROR1("Invalid minimum in config element descriptor %s",entry); - xbt_free(entrycpy); - xbt_abort(); - } + xbt_assert1(tok,"Invalid minimum in config element descriptor %s",entry); - if (!strcmp(tok,"_to_")){ - ERROR3("%s%s%s", - "Invalid config element descriptor: ",entry, - "; Should be :_to__"); - xbt_free(entrycpy); - xbt_abort(); - } + xbt_assert2(strcmp(tok,"_to_"), + "Invalid config element descriptor : %s%s", + entry, + "; Should be :_to__"); tok += strlen("_to_"); max=strtol(tok, &tok, 10); - if (!tok) { - ERROR1("Invalid maximum in config element descriptor %s",entry); - xbt_free(entrycpy); - xbt_abort(); - } + xbt_assert1(tok,"Invalid maximum in config element descriptor %s",entry); - if (*(tok++)!='_') { - ERROR3("%s%s%s", - "Invalid config element descriptor: ",entry, - "; Should be :_to__"); - xbt_free(entrycpy); - xbt_abort(); - } + xbt_assert2( *(tok++) =='_', + "Invalid config element descriptor: %s%s",entry, + "; Should be :_to__"); for (type=0; typecontent); - if (cell->min > size) { - ERROR4("Config elem %s needs at least %d %s, but there is only %d values.", + xbt_dict_foreach((xbt_dict_t)cfg,cursor,name,variable) { + size = xbt_dynar_length(variable->content); + if (variable->min > size) { + xbt_dict_cursor_free(&cursor); + THROW4(mismatch_error,0, + "Config elem %s needs at least %d %s, but there is only %d values.", name, - cell->min, - xbt_cfgelm_type_name[cell->type], + variable->min, + xbt_cfgelm_type_name[variable->type], size); - xbt_dict_cursor_free(&cursor); - return mismatch_error; } - if (cell->max < size) { - ERROR4("Config elem %s accepts at most %d %s, but there is %d values.", + if (variable->max > 0 && variable->max < size) { + xbt_dict_cursor_free(&cursor); + THROW4(mismatch_error,0, + "Config elem %s accepts at most %d %s, but there is %d values.", name, - cell->max, - xbt_cfgelm_type_name[cell->type], + variable->max, + xbt_cfgelm_type_name[variable->type], size); - xbt_dict_cursor_free(&cursor); - return mismatch_error; } - } xbt_dict_cursor_free(&cursor); - return no_error; } -static xbt_error_t xbt_cfgelm_get(xbt_cfg_t cfg, - const char *name, - e_xbt_cfgelm_type_t type, - /* OUT */ xbt_cfgelm_t *whereto){ - - xbt_error_t errcode = xbt_dict_get((xbt_dict_t)cfg,name, - (void**)whereto); - - if (errcode == mismatch_error) { - ERROR1("No registered cell %s in this config set", - name); - return mismatch_error; +static xbt_cfgelm_t xbt_cfgelm_get(xbt_cfg_t cfg, + const char *name, + e_xbt_cfgelm_type_t type){ + xbt_cfgelm_t res=NULL; + xbt_ex_t e; + + TRY { + res = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + if (e.category == not_found_error) { + xbt_ex_free(e); + THROW1(not_found_error,0, + "No registered variable '%s' in this config set",name); + } + RETHROW; } - if (errcode != no_error) - return errcode; - xbt_assert3((*whereto)->type == type, + xbt_assert3(type == xbt_cfgelm_any || res->type == type, "You tried to access to the config element %s as an %s, but its type is %s.", name, xbt_cfgelm_type_name[type], - xbt_cfgelm_type_name[(*whereto)->type]); + xbt_cfgelm_type_name[res->type]); - return no_error; + return res; } -/** - * xbt_cfg_get_type: +/** @brief Get the type of this variable in that configuration set * - * @cfg: the config set - * @name: the name of the element - * @type: the result + * \arg cfg the config set + * \arg name the name of the element + * \arg type the result * - * Give the type of the config element */ -xbt_error_t -xbt_cfg_get_type(xbt_cfg_t cfg, const char *name, - /* OUT */e_xbt_cfgelm_type_t *type) { - - xbt_cfgelm_t cell; - xbt_error_t errcode; +e_xbt_cfgelm_type_t +xbt_cfg_get_type(xbt_cfg_t cfg, const char *name) { - TRYCATCH(mismatch_error,xbt_dict_get((xbt_dict_t)cfg,name,(void**)&cell)); + xbt_cfgelm_t variable = NULL; + xbt_ex_t e; - if (errcode == mismatch_error) { - ERROR1("Can't get the type of '%s' since this cell does not exist", - name); - return mismatch_error; + TRY { + variable = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + if (e.category == not_found_error) { + xbt_ex_free(e); + THROW1(not_found_error,0, + "Can't get the type of '%s' since this variable does not exist",name); + } + RETHROW; } - *type=cell->type; + INFO1("type in variable = %d",variable->type); - return no_error; + return variable->type; } /*----[ Setting ]---------------------------------------------------------*/ -/** - * xbt_cfg_set_vargs(): - * @cfg: config set to fill - * @varargs: NULL-terminated list of pairs {(const char*)key, value} +/** @brief va_args version of xbt_cfg_set + * + * \arg cfg config set to fill + * \arg n variable name + * \arg pa variable value * * Add some values to the config set. - * @warning: if the list isn't NULL terminated, it will segfault. */ -xbt_error_t -xbt_cfg_set_vargs(xbt_cfg_t cfg, va_list pa) { - char *str,*name; +void +xbt_cfg_set_vargs(xbt_cfg_t cfg, const char *name, va_list pa) { + char *str; int i; double d; - e_xbt_cfgelm_type_t type; + e_xbt_cfgelm_type_t type=0; /* Set a dummy value to make gcc happy. It cannot get uninitialized */ - xbt_error_t errcode; + xbt_ex_t e; - while ((name=va_arg(pa,char *))) { - - if (!xbt_cfg_get_type(cfg,name,&type)) { - ERROR1("Can't set the property '%s' since it's not registered",name); - return mismatch_error; + TRY { + type = xbt_cfg_get_type(cfg,name); + } CATCH(e) { + if (e.category == not_found_error) { + xbt_ex_free(e); + THROW1(not_found_error,0,"Can't set the property '%s' since it's not registered",name); } + RETHROW; + } - switch (type) { - case xbt_cfgelm_host: - str = va_arg(pa, char *); - i=va_arg(pa,int); - TRY(xbt_cfg_set_host(cfg,name,str,i)); - break; + switch (type) { + case xbt_cfgelm_host: + str = va_arg(pa, char *); + i=va_arg(pa,int); + xbt_cfg_set_host(cfg,name,str,i); + break; - case xbt_cfgelm_string: - str=va_arg(pa, char *); - TRY(xbt_cfg_set_string(cfg, name, str)); - break; + case xbt_cfgelm_string: + str=va_arg(pa, char *); + xbt_cfg_set_string(cfg, name, str); + break; - case xbt_cfgelm_int: - i=va_arg(pa,int); - TRY(xbt_cfg_set_int(cfg,name,i)); - break; + case xbt_cfgelm_int: + i=va_arg(pa,int); + xbt_cfg_set_int(cfg,name,i); + break; - case xbt_cfgelm_double: - d=va_arg(pa,double); - TRY(xbt_cfg_set_double(cfg,name,d)); - break; + case xbt_cfgelm_double: + d=va_arg(pa,double); + xbt_cfg_set_double(cfg,name,d); + break; - default: - RAISE1(unknown_error,"Config element cell %s not valid.",name); - } + default: + xbt_assert2(0,"Config element variable %s not valid (type=%d)",name,type); } - return no_error; } -/** - * xbt_cfg_set(): - * @cfg: config set to fill - * @varargs: NULL-terminated list of pairs {(const char*)key, value} +/** @brief Add a NULL-terminated list of pairs {(char*)key, value} to the set + * + * \arg cfg config set to fill + * \arg name variable name + * \arg varargs variable value * - * Add some values to the config set. - * @warning: if the list isn't NULL terminated, it will segfault. */ -xbt_error_t xbt_cfg_set(xbt_cfg_t cfg, ...) { +void xbt_cfg_set(xbt_cfg_t cfg, const char *name, ...) { va_list pa; - xbt_error_t errcode; - va_start(pa,cfg); - errcode=xbt_cfg_set_vargs(cfg,pa); + va_start(pa,name); + xbt_cfg_set_vargs(cfg,name,pa); va_end(pa); - return errcode; } -/** - * xbt_cfg_set_parse(): - * @cfg: config set to fill - * @options: a string containing the content to add to the config set. This - * is a '\t',' ' or '\n' separated list of cells. Each individual cell is +/** @brief Add values parsed from a string into a config set + * + * \arg cfg config set to fill + * \arg options a string containing the content to add to the config set. This + * is a '\\t',' ' or '\\n' separated list of variables. Each individual variable is * like "[name]:[value]" where [name] is the name of an already registred - * cell, and [value] conforms to the data type under which this cell was + * variable, and [value] conforms to the data type under which this variable was * registred. * - * Add the cells described in a string to a config set. + * @todo This is a crude manual parser, it should be a proper lexer. */ -xbt_error_t +void xbt_cfg_set_parse(xbt_cfg_t cfg, const char *options) { + xbt_ex_t e; + int i; double d; char *str; - xbt_cfgelm_t cell; + xbt_cfgelm_t variable=NULL; char *optionlist_cpy; char *option, *name,*val; int len; - xbt_error_t errcode; XBT_IN; if (!options || !strlen(options)) { /* nothing to do */ - return no_error; + return; } optionlist_cpy=xbt_strdup(options); @@ -557,19 +533,21 @@ xbt_cfg_set_parse(xbt_cfg_t cfg, const char *options) { /* Pass the value */ while (option-name<=(len-1) && *option != ' ' && *option != '\n' && *option != '\t') { - /*fprintf(stderr,"Take %c.\n",*option);*/ + DEBUG1("Take %c.",*option); option++; } if (option-name == len) { - /*fprintf(stderr,"Boundary=EOL\n");*/ + DEBUG0("Boundary=EOL"); option=NULL; /* don't do next iteration */ } else { - /*fprintf(stderr,"Boundary on '%c'. len=%d;option-name=%d\n",*option,len,option-name);*/ + DEBUG3("Boundary on '%c'. len=%d;option-name=%d", + *option,len,option-name); /* Pass the following blank chars */ *(option++)='\0'; - while (option-name<(len-1) && (*option == ' ' || *option == '\n' || *option == '\t')) { + while (option-name<(len-1) && + (*option == ' ' || *option == '\n' || *option == '\t')) { /* fprintf(stderr,"Ignore a blank char.\n");*/ option++; } @@ -585,188 +563,203 @@ xbt_cfg_set_parse(xbt_cfg_t cfg, const char *options) { val=strchr(name,':'); if (!val) { - xbt_free(optionlist_cpy); + free(optionlist_cpy); xbt_assert1(FALSE, - "Malformated option: '%s'; Should be of the form 'name:value'", - name); + "Malformated option: '%s'; Should be of the form 'name:value'", + name); } *(val++)='\0'; DEBUG2("name='%s';val='%s'",name,val); - errcode=xbt_dict_get((xbt_dict_t)cfg,name,(void**)&cell); - switch (errcode) { - case no_error: - break; - case mismatch_error: - ERROR1("No registrated cell corresponding to '%s'.",name); - xbt_free(optionlist_cpy); - return mismatch_error; - break; - default: - xbt_free(optionlist_cpy); - return errcode; - } - - switch (cell->type) { - case xbt_cfgelm_string: - TRYCLEAN(xbt_cfg_set_string(cfg, name, val), - xbt_free(optionlist_cpy)); - break; - - case xbt_cfgelm_int: - i=strtol(val, &val, 0); - if (val==NULL) { - xbt_free(optionlist_cpy); - xbt_assert1(FALSE, - "Value of option %s not valid. Should be an integer", - name); - } - - TRYCLEAN(xbt_cfg_set_int(cfg,name,i), - xbt_free(optionlist_cpy)); - break; - - case xbt_cfgelm_double: - d=strtod(val, &val); - if (val==NULL) { - xbt_free(optionlist_cpy); - xbt_assert1(FALSE, - "Value of option %s not valid. Should be a double", - name); - } - - TRYCLEAN(xbt_cfg_set_double(cfg,name,d), - xbt_free(optionlist_cpy)); - break; - - case xbt_cfgelm_host: - str=val; - val=strchr(val,':'); - if (!val) { - xbt_free(optionlist_cpy); - xbt_assert1(FALSE, - "Value of option %s not valid. Should be an host (machine:port)", - name); + TRY { + variable = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + /* put it back on what won't get freed, ie within "options" and out of "optionlist_cpy" */ + name = (char*) (optionlist_cpy-name + options); + free(optionlist_cpy); + if (e.category == not_found_error) { + xbt_ex_free(e); + THROW1(not_found_error,0,"No registrated variable corresponding to '%s'.",name); } + RETHROW; + } - *(val++)='\0'; - i=strtol(val, &val, 0); - if (val==NULL) { - xbt_free(optionlist_cpy); - xbt_assert1(FALSE, - "Value of option %s not valid. Should be an host (machine:port)", - name); + TRY { + switch (variable->type) { + case xbt_cfgelm_string: + xbt_cfg_set_string(cfg, name, val); /* throws */ + break; + + case xbt_cfgelm_int: + i=strtol(val, &val, 0); + if (val==NULL) { + free(optionlist_cpy); + xbt_assert1(FALSE, + "Value of option %s not valid. Should be an integer", + name); + } + + xbt_cfg_set_int(cfg,name,i); /* throws */ + break; + + case xbt_cfgelm_double: + d=strtod(val, &val); + if (val==NULL) { + free(optionlist_cpy); + xbt_assert1(FALSE, + "Value of option %s not valid. Should be a double", + name); + } + + xbt_cfg_set_double(cfg,name,d); /* throws */ + break; + + case xbt_cfgelm_host: + str=val; + val=strchr(val,':'); + if (!val) { + free(optionlist_cpy); + xbt_assert1(FALSE, + "Value of option %s not valid. Should be an host (machine:port)", + name); + } + + *(val++)='\0'; + i=strtol(val, &val, 0); + if (val==NULL) { + free(optionlist_cpy); + xbt_assert1(FALSE, + "Value of option %s not valid. Should be an host (machine:port)", + name); + } + + xbt_cfg_set_host(cfg,name,str,i); /* throws */ + break; + + default: + THROW1(unknown_error,0,"Type of config element %s is not valid.",name); } - - TRYCLEAN(xbt_cfg_set_host(cfg,name,str,i), - xbt_free(optionlist_cpy)); - break; - - default: - xbt_free(optionlist_cpy); - RAISE1(unknown_error,"Type of config element %s is not valid.",name); + } CATCH(e) { + free(optionlist_cpy); + RETHROW; } - } - xbt_free(optionlist_cpy); - return no_error; + free(optionlist_cpy); + } -/** - * xbt_cfg_set_int: - * - * @cfg: the config set - * @name: the name of the cell - * @val: the value of the cell +/** @brief Set or add an integer value to \a name within \a cfg * - * Set the value of the cell @name in @cfg with the provided value. + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the value of the variable */ -xbt_error_t +void xbt_cfg_set_int(xbt_cfg_t cfg,const char*name, int val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; + xbt_cfgelm_t variable; VERB2("Configuration setting: %s=%d",name,val); - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_int); - if (cell->max > 1) { - xbt_dynar_push(cell->content,&val); + if (variable->max == 1) { + if (variable->cb_rm && xbt_dynar_length(variable->content)) + (*variable->cb_rm)(name, 0); + + xbt_dynar_set(variable->content,0,&val); } else { - xbt_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + THROW3(mismatch_error,0, + "Cannot add value %d to the config element %s since it's already full (size=%d)", + val,name,variable->max); + + xbt_dynar_push(variable->content,&val); } - return no_error; + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); } -/** - * xbt_cfg_set_double: - * @cfg: the config set - * @name: the name of the cell - * @val: the doule to set +/** @brief Set or add a double value to \a name within \a cfg * - * Set the value of the cell @name in @cfg with the provided value. + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the doule to set */ -xbt_error_t +void xbt_cfg_set_double(xbt_cfg_t cfg,const char*name, double val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; + xbt_cfgelm_t variable; VERB2("Configuration setting: %s=%f",name,val); - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_double); - if (cell->max > 1) { - xbt_dynar_push(cell->content,&val); + if (variable->max == 1) { + if (variable->cb_rm && xbt_dynar_length(variable->content)) + (*variable->cb_rm)(name, 0); + + xbt_dynar_set(variable->content,0,&val); } else { - xbt_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + THROW3(mismatch_error,0, + "Cannot add value %f to the config element %s since it's already full (size=%d)", + val,name,variable->max); + + xbt_dynar_push(variable->content,&val); } - return no_error; + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); } -/** - * xbt_cfg_set_string: +/** @brief Set or add a string value to \a name within \a cfg * - * @cfg: the config set - * @name: the name of the cell - * @val: the value to be added + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the value to be added * - * Set the value of the cell @name in @cfg with the provided value. */ -xbt_error_t +void xbt_cfg_set_string(xbt_cfg_t cfg,const char*name, const char*val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; + xbt_cfgelm_t variable; char *newval = xbt_strdup(val); VERB2("Configuration setting: %s=%s",name,val); - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_string); - if (cell->max > 1) { - xbt_dynar_push(cell->content,&newval); + if (variable->max == 1) { + if (variable->cb_rm && xbt_dynar_length(variable->content)) + (*variable->cb_rm)(name, 0); + + xbt_dynar_set(variable->content,0,&newval); } else { - xbt_dynar_set(cell->content,0,&newval); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + THROW3(mismatch_error,0, + "Cannot add value %s to the config element %s since it's already full (size=%d)", + name,val,variable->max); + + xbt_dynar_push(variable->content,&newval); } - return no_error; + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); } -/** - * xbt_cfg_set_host: +/** @brief Set or add an host value to \a name within \a cfg * - * @cfg: the config set - * @name: the name of the cell - * @host: the host - * @port: the port number + * \arg cfg the config set + * \arg name the name of the variable + * \arg host the host + * \arg port the port number * - * Set the value of the cell @name in @cfg with the provided value - * on the given @host to the given @port + * \e host values are composed of a string (hostname) and an integer (port) */ -xbt_error_t +void xbt_cfg_set_host(xbt_cfg_t cfg,const char*name, const char *host,int port) { - xbt_cfgelm_t cell; - xbt_error_t errcode; + xbt_cfgelm_t variable; xbt_host_t *val=xbt_new(xbt_host_t,1); VERB3("Configuration setting: %s=%s:%d",name,host,port); @@ -774,329 +767,375 @@ xbt_cfg_set_host(xbt_cfg_t cfg,const char*name, val->name = xbt_strdup(name); val->port = port; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_host); - if (cell->max > 1) { - xbt_dynar_push(cell->content,&val); + if (variable->max == 1) { + if (variable->cb_rm && xbt_dynar_length(variable->content)) + (*variable->cb_rm)(name, 0); + + xbt_dynar_set(variable->content,0,&val); } else { - xbt_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + THROW4(mismatch_error,0, + "Cannot add value %s:%d to the config element %s since it's already full (size=%d)", + host,port,name,variable->max); + + xbt_dynar_push(variable->content,&val); } - return no_error; + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); } /* ---- [ Removing ] ---- */ -/** - * xbt_cfg_rm_int: - * - * @cfg: the config set - * @name: the name of the cell - * @val: the value to be removed +/** @brief Remove the provided \e val integer value from a variable * - * Remove the provided @val from the cell @name in @cfg. + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the value to be removed */ -xbt_error_t xbt_cfg_rm_int(xbt_cfg_t cfg,const char*name, int val) { +void xbt_cfg_rm_int(xbt_cfg_t cfg,const char*name, int val) { - xbt_cfgelm_t cell; + xbt_cfgelm_t variable; int cpt,seen; - xbt_error_t errcode; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_int); - xbt_dynar_foreach(cell->content,cpt,seen) { + if (xbt_dynar_length(variable->content) == variable->min) + THROW3(mismatch_error,0, + "Cannot remove value %d from the config element %s since it's already at its minimal size (=%d)", + val,name,variable->min); + + xbt_dynar_foreach(variable->content,cpt,seen) { if (seen == val) { - xbt_dynar_cursor_rm(cell->content,&cpt); - return no_error; + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); + return; } } - ERROR2("Can't remove the value %d of config element %s: value not found.", - val,name); - return mismatch_error; + THROW2(not_found_error,0, + "Can't remove the value %d of config element %s: value not found.",val,name); } -/** - * xbt_cfg_rm_double: - * - * @cfg: the config set - * @name: the name of the cell - * @val: the value to be removed +/** @brief Remove the provided \e val double value from a variable * - * Remove the provided @val from the cell @name in @cfg. + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the value to be removed */ -xbt_error_t xbt_cfg_rm_double(xbt_cfg_t cfg,const char*name, double val) { - xbt_cfgelm_t cell; +void xbt_cfg_rm_double(xbt_cfg_t cfg,const char*name, double val) { + xbt_cfgelm_t variable; int cpt; double seen; - xbt_error_t errcode; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_double); - xbt_dynar_foreach(cell->content,cpt,seen) { + if (xbt_dynar_length(variable->content) == variable->min) + THROW3(mismatch_error,0, + "Cannot remove value %f from the config element %s since it's already at its minimal size (=%d)", + val,name,variable->min); + + xbt_dynar_foreach(variable->content,cpt,seen) { if (seen == val) { - xbt_dynar_cursor_rm(cell->content,&cpt); - return no_error; + xbt_dynar_cursor_rm(variable->content,&cpt); + if (variable->cb_rm) + (*variable->cb_rm)(name, cpt); + return; } } - ERROR2("Can't remove the value %f of config element %s: value not found.", - val,name); - return mismatch_error; + THROW2(not_found_error,0, + "Can't remove the value %f of config element %s: value not found.",val,name); } -/** - * xbt_cfg_rm_string: +/** @brief Remove the provided \e val string value from a variable * - * @cfg: the config set - * @name: the name of the cell - * @val: the value of the string which will be removed - * - * Remove the provided @val from the cell @name in @cfg. + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the value of the string which will be removed */ -xbt_error_t +void xbt_cfg_rm_string(xbt_cfg_t cfg,const char*name, const char *val) { - xbt_cfgelm_t cell; + xbt_cfgelm_t variable; int cpt; char *seen; - xbt_error_t errcode; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_string); - xbt_dynar_foreach(cell->content,cpt,seen) { + if (xbt_dynar_length(variable->content) == variable->min) + THROW3(mismatch_error,0, + "Cannot remove value %s from the config element %s since it's already at its minimal size (=%d)", + name,val,variable->min); + + xbt_dynar_foreach(variable->content,cpt,seen) { if (!strcpy(seen,val)) { - xbt_dynar_cursor_rm(cell->content,&cpt); - return no_error; + if (variable->cb_rm) + (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); + return; } } - ERROR2("Can't remove the value %s of config element %s: value not found.", - val,name); - return mismatch_error; + THROW2(not_found_error,0, + "Can't remove the value %s of config element %s: value not found.",val,name); } -/** - * xbt_cfg_rm_host: +/** @brief Remove the provided \e val host value from a variable * - * @cfg: the config set - * @name: the name of the cell - * @host: the hostname - * @port: the port number - * - * Remove the provided @host:@port from the cell @name in @cfg. + * \arg cfg the config set + * \arg name the name of the variable + * \arg host the hostname + * \arg port the port number */ -xbt_error_t +void xbt_cfg_rm_host(xbt_cfg_t cfg,const char*name, const char *host,int port) { - xbt_cfgelm_t cell; + xbt_cfgelm_t variable; int cpt; xbt_host_t *seen; - xbt_error_t errcode; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_host); - xbt_dynar_foreach(cell->content,cpt,seen) { + if (xbt_dynar_length(variable->content) == variable->min) + THROW4(mismatch_error,0, + "Cannot remove value %s:%d from the config element %s since it's already at its minimal size (=%d)", + host,port,name,variable->min); + + xbt_dynar_foreach(variable->content,cpt,seen) { if (!strcpy(seen->name,host) && seen->port == port) { - xbt_dynar_cursor_rm(cell->content,&cpt); - return no_error; + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); + return; } } - ERROR3("Can't remove the value %s:%d of config element %s: value not found.", + THROW3(not_found_error,0, + "Can't remove the value %s:%d of config element %s: value not found.", host,port,name); - return mismatch_error; } -/* rm everything */ +/** @brief Remove the \e pos th value from the provided variable */ -/** - * xbt_cfg_empty: +void xbt_cfg_rm_at (xbt_cfg_t cfg, const char *name, int pos) { + + xbt_cfgelm_t variable; + + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_any); + + if (xbt_dynar_length(variable->content) == variable->min) + THROW3(mismatch_error,0, + "Cannot remove %dth value from the config element %s since it's already at its minimal size (=%d)", + pos,name,variable->min); + + if (variable->cb_rm) (*variable->cb_rm)(name, pos); + xbt_dynar_remove_at(variable->content, pos, NULL); +} + +/** @brief Remove all the values from a variable * - * @cfg: the config set - * @name: the name of the cell - * - * rm evenything + * \arg cfg the config set + * \arg name the name of the variable */ -xbt_error_t +void xbt_cfg_empty(xbt_cfg_t cfg,const char*name) { - xbt_cfgelm_t cell; - - xbt_error_t errcode; - - TRYCATCH(mismatch_error, - xbt_dict_get((xbt_dict_t)cfg,name,(void**)&cell)); - if (errcode == mismatch_error) { - ERROR1("Can't empty '%s' since this config element does not exist", - name); - return mismatch_error; + xbt_cfgelm_t variable=NULL; + xbt_ex_t e; + + TRY { + variable = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + if (e.category != not_found_error) + RETHROW; + + xbt_ex_free(e); + THROW1(not_found_error,0, + "Can't empty '%s' since this config element does not exist", name); } - if (cell) { - xbt_dynar_reset(cell->content); + if (variable) { + if (variable->cb_rm) { + int cpt; + void *ignored; + xbt_dynar_foreach(variable->content,cpt,ignored) { + (*variable->cb_rm)(name, cpt); + } + } + xbt_dynar_reset(variable->content); } - return no_error; } /*----[ Getting ]---------------------------------------------------------*/ -/** - * xbt_cfg_get_int: - * @cfg: the config set - * @name: the name of the cell - * @val: the wanted value +/** @brief Retrieve an integer value of a variable (get a warning if not uniq) + * + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the wanted value * * Returns the first value from the config set under the given name. - * If there is more than one value, it will issue a warning. Consider using xbt_cfg_get_dynar() - * instead. + * If there is more than one value, it will issue a warning. Consider using + * xbt_cfg_get_dynar() instead. * - * @warning the returned value is the actual content of the config set + * \warning the returned value is the actual content of the config set */ -xbt_error_t -xbt_cfg_get_int (xbt_cfg_t cfg, - const char *name, - int *val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; - - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&cell)); +int +xbt_cfg_get_int (xbt_cfg_t cfg, const char *name) { + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_int); - if (xbt_dynar_length(cell->content) > 1) { + if (xbt_dynar_length(variable->content) > 1) { WARN2("You asked for the first value of the config element '%s', but there is %lu values", - name, xbt_dynar_length(cell->content)); + name, xbt_dynar_length(variable->content)); } - *val = xbt_dynar_get_as(cell->content, 0, int); - return no_error; + return xbt_dynar_get_as(variable->content, 0, int); } -/** - * xbt_cfg_get_double: - * @cfg: the config set - * @name: the name of the cell - * @val: the wanted value +/** @brief Retrieve a double value of a variable (get a warning if not uniq) + * + * \arg cfg the config set + * \arg name the name of the variable + * \arg val the wanted value * * Returns the first value from the config set under the given name. - * If there is more than one value, it will issue a warning. Consider using xbt_cfg_get_dynar() - * instead. + * If there is more than one value, it will issue a warning. Consider using + * xbt_cfg_get_dynar() instead. * - * @warning the returned value is the actual content of the config set + * \warning the returned value is the actual content of the config set */ -xbt_error_t -xbt_cfg_get_double(xbt_cfg_t cfg, - const char *name, - double *val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; +double +xbt_cfg_get_double(xbt_cfg_t cfg, const char *name) { + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_double); - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&cell)); - - if (xbt_dynar_length(cell->content) > 1) { + if (xbt_dynar_length(variable->content) > 1) { WARN2("You asked for the first value of the config element '%s', but there is %lu values\n", - name, xbt_dynar_length(cell->content)); + name, xbt_dynar_length(variable->content)); } - *val = xbt_dynar_get_as(cell->content, 0, double); - return no_error; + return xbt_dynar_get_as(variable->content, 0, double); } -/** - * xbt_cfg_get_string: +/** @brief Retrieve a string value of a variable (get a warning if not uniq) * - * @th: the config set - * @name: the name of the cell - * @val: the wanted value + * \arg th the config set + * \arg name the name of the variable + * \arg val the wanted value * * Returns the first value from the config set under the given name. - * If there is more than one value, it will issue a warning. Consider using xbt_cfg_get_dynar() - * instead. + * If there is more than one value, it will issue a warning. Consider using + * xbt_cfg_get_dynar() instead. * - * @warning the returned value is the actual content of the config set + * \warning the returned value is the actual content of the config set */ -xbt_error_t xbt_cfg_get_string(xbt_cfg_t cfg, - const char *name, - char **val) { - xbt_cfgelm_t cell; - xbt_error_t errcode; - - *val=NULL; +char* xbt_cfg_get_string(xbt_cfg_t cfg, const char *name) { + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_string); - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&cell)); - - if (xbt_dynar_length(cell->content) > 1) { + if (xbt_dynar_length(variable->content) > 1) { WARN2("You asked for the first value of the config element '%s', but there is %lu values\n", - name, xbt_dynar_length(cell->content)); + name, xbt_dynar_length(variable->content)); } - *val = xbt_dynar_get_as(cell->content, 0, char *); - return no_error; + return xbt_dynar_get_as(variable->content, 0, char *); } -/** - * xbt_cfg_get_host: +/** @brief Retrieve an host value of a variable (get a warning if not uniq) * - * @cfg: the config set - * @name: the name of the cell - * @host: the host - * @port: the port number + * \arg cfg the config set + * \arg name the name of the variable + * \arg host the host + * \arg port the port number * * Returns the first value from the config set under the given name. - * If there is more than one value, it will issue a warning. Consider using xbt_cfg_get_dynar() - * instead. + * If there is more than one value, it will issue a warning. Consider using + * xbt_cfg_get_dynar() instead. * - * @warning the returned value is the actual content of the config set + * \warning the returned value is the actual content of the config set */ -xbt_error_t xbt_cfg_get_host (xbt_cfg_t cfg, - const char *name, - char **host, - int *port) { - xbt_cfgelm_t cell; - xbt_error_t errcode; +void xbt_cfg_get_host (xbt_cfg_t cfg, const char *name, + char **host, int *port) { + xbt_cfgelm_t variable; xbt_host_t *val; - TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&cell)); + variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_host); - if (xbt_dynar_length(cell->content) > 1) { + if (xbt_dynar_length(variable->content) > 1) { WARN2("You asked for the first value of the config element '%s', but there is %lu values\n", - name, xbt_dynar_length(cell->content)); + name, xbt_dynar_length(variable->content)); } - val = xbt_dynar_get_as(cell->content, 0, xbt_host_t*); + val = xbt_dynar_get_as(variable->content, 0, xbt_host_t*); *host=val->name; *port=val->port; - - return no_error; } -/** - * xbt_cfg_get_dynar: - * @cfg: where to search in - * @name: what to search for - * @dynar: result +/** @brief Retrieve the dynar of all the values stored in a variable + * + * \arg cfg where to search in + * \arg name what to search for + * \arg dynar result * - * Get the data stored in the config bag. + * Get the data stored in the config set. * - * @warning the returned value is the actual content of the config set + * \warning the returned value is the actual content of the config set */ -xbt_error_t xbt_cfg_get_dynar (xbt_cfg_t cfg, - const char *name, - xbt_dynar_t *dynar) { - xbt_cfgelm_t cell; - xbt_error_t errcode = xbt_dict_get((xbt_dict_t)cfg,name, - (void**)&cell); - - if (errcode == mismatch_error) { - ERROR1("No registered cell %s in this config set", - name); - return mismatch_error; +xbt_dynar_t xbt_cfg_get_dynar (xbt_cfg_t cfg, const char *name) { + xbt_cfgelm_t variable=NULL; + xbt_ex_t e; + + TRY { + variable = xbt_dict_get((xbt_dict_t)cfg,name); + } CATCH(e) { + if (e.category == not_found_error) { + xbt_ex_free(e); + THROW1(not_found_error,0, + "No registered variable %s in this config set",name); + } + RETHROW; } - if (errcode != no_error) - return errcode; - *dynar = cell->content; - return no_error; + return variable->content; } + +/** @brief Retrieve one of the integer value of a variable */ +int +xbt_cfg_get_int_at(xbt_cfg_t cfg, const char *name, int pos) { + + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_int); + return xbt_dynar_get_as(variable->content, pos, int); +} + +/** @brief Retrieve one of the double value of a variable */ +double +xbt_cfg_get_double_at(xbt_cfg_t cfg, const char *name, int pos) { + + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_double); + return xbt_dynar_get_as(variable->content, pos, double); +} + + +/** @brief Retrieve one of the string value of a variable */ +char* +xbt_cfg_get_string_at(xbt_cfg_t cfg, const char *name, int pos) { + + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_string); + return xbt_dynar_get_as(variable->content, pos, char*); +} + +/** @brief Retrieve one of the host value of a variable */ +void +xbt_cfg_get_host_at(xbt_cfg_t cfg, const char *name, int pos, + char **host, int *port) { + + xbt_cfgelm_t variable = xbt_cfgelm_get(cfg,name,xbt_cfgelm_int); + xbt_host_t *val = xbt_dynar_get_ptr(variable->content, pos); + + *port = val->port; + *host = val->name; +}