X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/b1acf7fe2f1886e9adcb4313f53548c9995a906c..89f2a9326028a243896fa4d4fbe1475bea9960c3:/src/xbt/config.c diff --git a/src/xbt/config.c b/src/xbt/config.c index b10cd74bdc..aade80293a 100644 --- a/src/xbt/config.c +++ b/src/xbt/config.c @@ -1,53 +1,60 @@ /* $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 "gras_private.h" /* prototypes of this module */ +#include "xbt/misc.h" +#include "xbt/sysdep.h" +#include "xbt/log.h" +#include "xbt/error.h" +#include "xbt/dynar.h" +#include "xbt/dict.h" -GRAS_LOG_NEW_DEFAULT_CATEGORY(config); +#include "xbt/config.h" /* prototypes of this module */ -/* gras_cfgelm_t: the typedef corresponding to a config cell. +XBT_LOG_NEW_DEFAULT_SUBCATEGORY(config,xbt,"configuration support"); + +/* 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 */ - gras_cfgelm_type_t type; + /* 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) */ - gras_dynar_t *content; -} gras_cfgelm_t; + xbt_dynar_t content; +} s_xbt_cfgelm_t,*xbt_cfgelm_t; -static const char *gras_cfgelm_type_name[gras_cfgelm_type_count]= - {"int","double","string","host"}; +static const char *xbt_cfgelm_type_name[xbt_cfgelm_type_count]= + {"int","double","string","host","any"}; -/* Internal stuff used in cache to free a cell */ -static void gras_cfgelm_free(void *data); +/* Internal stuff used in cache to free a variable */ +static void xbt_cfgelm_free(void *data); -/* Retrieve the cell we'll modify */ -static gras_error_t gras_cfgelm_get(gras_cfg_t *cfg, const char *name, - gras_cfgelm_type_t type, - /* OUT */ gras_cfgelm_t **whereto); +/* Retrieve the variable 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 gras_cfg_str_free(void *d); -void gras_cfg_host_free(void *d); - -void gras_cfg_str_free(void *d){ +static void xbt_cfg_str_free(void *d){ free(*(void**)d); } -void gras_cfg_host_free(void *d){ - gras_host_t *h=(gras_host_t*) *(void**)d; +static void xbt_cfg_host_free(void *d){ + xbt_host_t *h=(xbt_host_t*) *(void**)d; if (h) { if (h->name) free(h->name); free(h); @@ -56,118 +63,101 @@ void gras_cfg_host_free(void *d){ /*----[ Memory management ]-----------------------------------------------*/ -/** - * gras_cfg_new: - * - * @whereto: +/** @brief Constructor * * Initialise an config set */ -gras_error_t gras_cfg_new(gras_cfg_t **whereto) { - return gras_dict_new((gras_dict_t**)whereto); +xbt_cfg_t xbt_cfg_new(void) { + return (xbt_cfg_t)xbt_dict_new(); } -/** - * gras_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 */ -gras_error_t -gras_cfg_cpy(gras_cfg_t **whereto, gras_cfg_t *tocopy) { - gras_dict_cursor_t *cursor=NULL; - gras_cfgelm_t *cell=NULL; +void +xbt_cfg_cpy(xbt_cfg_t tocopy,xbt_cfg_t *whereto) { + xbt_dict_cursor_t cursor=NULL; + xbt_cfgelm_t variable=NULL; char *name=NULL; - int errcode=no_error; *whereto=NULL; - gras_assert0(tocopy,"cannot copy NULL config"); + xbt_assert0(tocopy,"cannot copy NULL config"); - gras_dict_foreach((gras_dict_t*)tocopy,cursor,name,cell) { - if ((errcode=gras_cfg_register(*whereto, name, cell->type, - cell->min, cell->max)) != no_error) { - if (cursor) gras_dict_cursor_free(cursor); - if (*whereto) gras_cfg_free(whereto); - return errcode; - } + 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); } - - return errcode; } -void gras_cfg_free(gras_cfg_t **cfg) { - gras_dict_free((gras_dict_t**)cfg); +/** @brief Destructor */ +void xbt_cfg_free(xbt_cfg_t *cfg) { + xbt_dict_free((xbt_dict_t*)cfg); } -/** - * gras_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 -gras_cfg_dump(const char *name,const char *indent,gras_cfg_t *cfg) { - gras_dict_t *dict = (gras_dict_t*) cfg; - gras_dict_cursor_t *cursor=NULL; - gras_cfgelm_t *cell=NULL; +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 variable=NULL; char *key=NULL; int i; - gras_error_t errcode; int size; int ival; char *sval; double dval; - gras_host_t hval; + xbt_host_t *hval; if (name) printf("%s>> Dumping of the config set '%s':\n",indent,name); - gras_dict_foreach(dict,cursor,key,cell) { - if ((errcode=gras_dict_cursor_get_key(cursor,(char**)&name)) != no_error || - (errcode=gras_dict_cursor_get_data(cursor,(void**)&cell)) != no_error) { - gras_dict_cursor_free(cursor); - return; - } + xbt_dict_foreach(dict,cursor,key,variable) { printf("%s %s:",indent,key); - fflush(stdout); - size = gras_dynar_length(cell->content); - printf("%d_to_%d_%s. Actual size=%d. List of values:\n", - cell->min,cell->max,gras_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) { - case gras_cfgelm_int: + switch (variable->type) { + + case xbt_cfgelm_int: for (i=0; icontent,i,&ival); + ival = xbt_dynar_get_as(variable->content,i,int); printf ("%s %d\n",indent,ival); } break; - case gras_cfgelm_double: + case xbt_cfgelm_double: for (i=0; icontent,i,&dval); + dval = xbt_dynar_get_as(variable->content,i,double); printf ("%s %f\n",indent,dval); } break; - case gras_cfgelm_string: + case xbt_cfgelm_string: for (i=0; icontent,i,&sval); + sval = xbt_dynar_get_as(variable->content,i,char*); printf ("%s %s\n",indent,sval); } break; - case gras_cfgelm_host: + case xbt_cfgelm_host: for (i=0; icontent,i,&hval); - printf ("%s %s:%d\n",indent,hval.name,hval.port); + hval = xbt_dynar_get_as(variable->content,i,xbt_host_t*); + printf ("%s %s:%d\n",indent,hval->name,hval->port); } break; @@ -178,118 +168,117 @@ gras_cfg_dump(const char *name,const char *indent,gras_cfg_t *cfg) { } if (name) printf("%s<< End of the config set '%s'\n",indent,name); + fflush(stdout); - gras_dict_cursor_free(cursor); + xbt_dict_cursor_free(&cursor); return; } -/** - * gras_cfgelm_free: - * - * @data: the data to be freed (typed as void* to be usable as free funct in dict) - * +/* * free an config element */ -void gras_cfgelm_free(void *data) { - gras_cfgelm_t *c=(gras_cfgelm_t *)data; +void xbt_cfgelm_free(void *data) { + xbt_cfgelm_t c=(xbt_cfgelm_t)data; if (!c) return; - gras_dynar_free(c->content); + xbt_dynar_free(&(c->content)); free(c); } /*----[ Registering stuff ]-----------------------------------------------*/ -/** - * gras_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 */ -gras_error_t -gras_cfg_register(gras_cfg_t *cfg, - const char *name, gras_cfgelm_type_t type, - int min, int max){ - gras_cfgelm_t *res; - gras_error_t errcode; - - TRYCATCH(mismatch_error,gras_dict_retrieve((gras_dict_t*)cfg,name,(void**)&res)); - - if (errcode != mismatch_error) { +void +xbt_cfg_register(xbt_cfg_t cfg, + 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; + + 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); + errcode = xbt_dict_get((xbt_dict_t)cfg,name,(void**)&res); + + if (errcode == no_error) { WARN1("Config elem %s registered twice.",name); /* Will be removed by the insertion of the new one */ } + xbt_assert_error(mismatch_error); - res=malloc(sizeof(gras_cfgelm_t)); - if (!res) - RAISE_MALLOC; + 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 gras_cfgelm_int: - TRY(gras_dynar_new(&(res->content), sizeof(int), NULL)); + case xbt_cfgelm_int: + res->content = xbt_dynar_new(sizeof(int), NULL); break; - case gras_cfgelm_double: - TRY(gras_dynar_new(&(res->content), sizeof(double), NULL)); + case xbt_cfgelm_double: + res->content = xbt_dynar_new(sizeof(double), NULL); break; - case gras_cfgelm_string: - TRY(gras_dynar_new(&(res->content),sizeof(char*),&gras_cfg_str_free)); + case xbt_cfgelm_string: + res->content = xbt_dynar_new(sizeof(char*),&xbt_cfg_str_free); break; - case gras_cfgelm_host: - TRY(gras_dynar_new(&(res->content),sizeof(gras_host_t*),&gras_cfg_host_free)); + case xbt_cfgelm_host: + res->content = xbt_dynar_new(sizeof(xbt_host_t*),&xbt_cfg_host_free); break; default: ERROR1("%d is an invalide type code",type); } - return gras_dict_insert((gras_dict_t*)cfg,name,res,&gras_cfgelm_free); + xbt_dict_set((xbt_dict_t)cfg,name,res,&xbt_cfgelm_free); } -/** - * gras_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. */ -gras_error_t -gras_cfg_unregister(gras_cfg_t *cfg,const char *name) { - return gras_dict_remove((gras_dict_t*)cfg,name); +xbt_error_t +xbt_cfg_unregister(xbt_cfg_t cfg,const char *name) { + return xbt_dict_remove((xbt_dict_t)cfg,name); } /** - * gras_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'. */ -gras_error_t -gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { - char *entrycpy=strdup(entry); +xbt_error_t +xbt_cfg_register_str(xbt_cfg_t cfg,const char *entry) { + char *entrycpy=xbt_strdup(entry); char *tok; int min,max; - gras_cfgelm_type_t type; - - gras_error_t errcode; + e_xbt_cfgelm_type_t type; tok=strchr(entrycpy, ':'); if (!tok) { @@ -297,7 +286,7 @@ gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { "Invalid config element descriptor: ",entry, "; Should be :_to__"); free(entrycpy); - gras_abort(); + xbt_abort(); } *(tok++)='\0'; @@ -305,7 +294,7 @@ gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { if (!tok) { ERROR1("Invalid minimum in config element descriptor %s",entry); free(entrycpy); - gras_abort(); + xbt_abort(); } if (!strcmp(tok,"_to_")){ @@ -313,7 +302,7 @@ gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { "Invalid config element descriptor: ",entry, "; Should be :_to__"); free(entrycpy); - gras_abort(); + xbt_abort(); } tok += strlen("_to_"); @@ -321,7 +310,7 @@ gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { if (!tok) { ERROR1("Invalid maximum in config element descriptor %s",entry); free(entrycpy); - gras_abort(); + xbt_abort(); } if (*(tok++)!='_') { @@ -329,225 +318,222 @@ gras_cfg_register_str(gras_cfg_t *cfg,const char *entry) { "Invalid config element descriptor: ",entry, "; Should be :_to__"); free(entrycpy); - gras_abort(); + xbt_abort(); } for (type=0; - typecontent); - if (cell->min > size) { + xbt_dict_foreach((xbt_dict_t)cfg,cursor,name,variable) { + size = xbt_dynar_length(variable->content); + if (variable->min > size) { ERROR4("Config elem %s needs at least %d %s, but there is only %d values.", name, - cell->min, - gras_cfgelm_type_name[cell->type], - size); + variable->min, + xbt_cfgelm_type_name[variable->type], + size); + xbt_dict_cursor_free(&cursor); return mismatch_error; } - if (cell->max < size) { + if (variable->max > 0 && variable->max < size) { ERROR4("Config elem %s accepts at most %d %s, but there is %d values.", name, - cell->max, - gras_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 gras_error_t gras_cfgelm_get(gras_cfg_t *cfg, +static xbt_error_t xbt_cfgelm_get(xbt_cfg_t cfg, const char *name, - gras_cfgelm_type_t type, - /* OUT */ gras_cfgelm_t **whereto){ - gras_error_t errcode; - - TRYCATCH(mismatch_error,gras_dict_retrieve((gras_dict_t*)cfg,name,(void**)whereto)); + 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", + RAISE1(mismatch_error, + "No registered variable '%s' in this config set", name); - return mismatch_error; } + if (errcode != no_error) + return errcode; - gras_assert3((*whereto)->type == type, + xbt_assert3(type == xbt_cfgelm_any || (*whereto)->type == type, "You tried to access to the config element %s as an %s, but its type is %s.", name, - gras_cfgelm_type_name[type], - gras_cfgelm_type_name[(*whereto)->type]); + xbt_cfgelm_type_name[type], + xbt_cfgelm_type_name[(*whereto)->type]); return no_error; } -/** - * gras_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 */ -gras_error_t -gras_cfg_get_type(gras_cfg_t *cfg, const char *name, - /* OUT */gras_cfgelm_type_t *type) { +xbt_error_t +xbt_cfg_get_type(xbt_cfg_t cfg, const char *name, + /* OUT */e_xbt_cfgelm_type_t *type) { - gras_cfgelm_t *cell; - gras_error_t errcode; + xbt_cfgelm_t variable; + xbt_error_t errcode; - TRYCATCH(mismatch_error,gras_dict_retrieve((gras_dict_t*)cfg,name,(void**)&cell)); + errcode=xbt_dict_get((xbt_dict_t)cfg,name,(void**)&variable); if (errcode == mismatch_error) { - ERROR1("Can't get the type of '%s' since this cell does not exist", + RAISE1(mismatch_error,"Can't get the type of '%s' since this variable does not exist", name); - return mismatch_error; + } else if (errcode != no_error) { + return errcode; } - *type=cell->type; + INFO1("type in variable = %d",variable->type); + *type=variable->type; return no_error; } /*----[ Setting ]---------------------------------------------------------*/ -/** - * gras_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. */ -gras_error_t -gras_cfg_set_vargs(gras_cfg_t *cfg, va_list pa) { - char *str,*name; +xbt_error_t +xbt_cfg_set_vargs(xbt_cfg_t cfg, const char *name, va_list pa) { + char *str; int i; double d; - gras_cfgelm_type_t type; + e_xbt_cfgelm_type_t type; - gras_error_t errcode; + xbt_error_t errcode; - while ((name=va_arg(pa,char *))) { - - if (!gras_cfg_get_type(cfg,name,&type)) { - ERROR1("Can't set the property '%s' since it's not registered",name); - return mismatch_error; - } + errcode = xbt_cfg_get_type(cfg,name,&type); + if (errcode != no_error) { + ERROR1("Can't set the property '%s' since it's not registered",name); + return mismatch_error; + } - switch (type) { - case gras_cfgelm_host: - str = va_arg(pa, char *); - i=va_arg(pa,int); - TRY(gras_cfg_set_host(cfg,name,str,i)); - break; + 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; - case gras_cfgelm_string: - str=va_arg(pa, char *); - TRY(gras_cfg_set_string(cfg, name, str)); - break; + case xbt_cfgelm_string: + str=va_arg(pa, char *); + TRY(xbt_cfg_set_string(cfg, name, str)); + break; - case gras_cfgelm_int: - i=va_arg(pa,int); - TRY(gras_cfg_set_int(cfg,name,i)); - break; + case xbt_cfgelm_int: + i=va_arg(pa,int); + TRY(xbt_cfg_set_int(cfg,name,i)); + break; - case gras_cfgelm_double: - d=va_arg(pa,double); - TRY(gras_cfg_set_double(cfg,name,d)); - break; + case xbt_cfgelm_double: + d=va_arg(pa,double); + TRY(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; } -/** - * gras_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. */ -gras_error_t gras_cfg_set(gras_cfg_t *cfg, ...) { +xbt_error_t xbt_cfg_set(xbt_cfg_t cfg, const char *name, ...) { va_list pa; - gras_error_t errcode; + xbt_error_t errcode; - va_start(pa,cfg); - errcode=gras_cfg_set_vargs(cfg,pa); + va_start(pa,name); + errcode=xbt_cfg_set_vargs(cfg,name,pa); va_end(pa); return errcode; } -/** - * gras_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. */ -gras_error_t -gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { +xbt_error_t +xbt_cfg_set_parse(xbt_cfg_t cfg, const char *options) { int i; double d; char *str; - gras_cfgelm_t *cell; + xbt_cfgelm_t variable; char *optionlist_cpy; char *option, *name,*val; int len; - gras_error_t errcode; + xbt_error_t errcode; + XBT_IN; if (!options || !strlen(options)) { /* nothing to do */ return no_error; } - optionlist_cpy=strdup(options); + optionlist_cpy=xbt_strdup(options); - DEBUG1("Options list='%s'",options); + DEBUG1("List to parse and set:'%s'",options); option=optionlist_cpy; while (1) { /* breaks in the code */ @@ -555,30 +541,33 @@ gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { break; name=option; len=strlen(name); - DEBUG3("Parse list '%s'. len=%d; option-name=%d",name,len,option-name); + DEBUG3("Still to parse and set: '%s'. len=%d; option-name=%ld", + name,len,(long)(option-name)); /* 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')) { - // fprintf(stderr,"Ignore a blank char.\n"); + while (option-name<(len-1) && + (*option == ' ' || *option == '\n' || *option == '\t')) { + /* fprintf(stderr,"Ignore a blank char.\n");*/ option++; } if (option-name == len-1) option=NULL; /* don't do next iteration */ } - DEBUG2("this='%s';rest='%s'",name,option); + DEBUG2("parse now:'%s'; parse later:'%s'",name,option); if (name[0] == ' ' || name[0] == '\n' || name[0] == '\t') continue; @@ -588,20 +577,20 @@ gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { val=strchr(name,':'); if (!val) { free(optionlist_cpy); - gras_assert1(FALSE, - "Malformated option: '%s'; Should be of the form 'name:value'", - name); + xbt_assert1(FALSE, + "Malformated option: '%s'; Should be of the form 'name:value'", + name); } *(val++)='\0'; DEBUG2("name='%s';val='%s'",name,val); - errcode=gras_dict_retrieve((gras_dict_t*)cfg,name,(void**)&cell); + errcode=xbt_dict_get((xbt_dict_t)cfg,name,(void**)&variable); switch (errcode) { case no_error: break; case mismatch_error: - ERROR1("No registrated cell corresponding to '%s'.",name); + ERROR1("No registrated variable corresponding to '%s'.",name); free(optionlist_cpy); return mismatch_error; break; @@ -610,45 +599,54 @@ gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { return errcode; } - switch (cell->type) { - case gras_cfgelm_string: - TRYCLEAN(gras_cfg_set_string(cfg, name, val), - free(optionlist_cpy)); + switch (variable->type) { + case xbt_cfgelm_string: + errcode = xbt_cfg_set_string(cfg, name, val); + if (errcode != no_error) { + free(optionlist_cpy); + return errcode; + } break; - case gras_cfgelm_int: + case xbt_cfgelm_int: i=strtol(val, &val, 0); if (val==NULL) { free(optionlist_cpy); - gras_assert1(FALSE, + xbt_assert1(FALSE, "Value of option %s not valid. Should be an integer", name); } - TRYCLEAN(gras_cfg_set_int(cfg,name,i), - free(optionlist_cpy)); + errcode = xbt_cfg_set_int(cfg,name,i); + if (errcode != no_error) { + free(optionlist_cpy); + return errcode; + } break; - case gras_cfgelm_double: + case xbt_cfgelm_double: d=strtod(val, &val); if (val==NULL) { free(optionlist_cpy); - gras_assert1(FALSE, + xbt_assert1(FALSE, "Value of option %s not valid. Should be a double", name); } - TRYCLEAN(gras_cfg_set_double(cfg,name,d), - free(optionlist_cpy)); + errcode = xbt_cfg_set_double(cfg,name,d); + if (errcode != no_error) { + free(optionlist_cpy); + return errcode; + } break; - case gras_cfgelm_host: + case xbt_cfgelm_host: str=val; val=strchr(val,':'); if (!val) { free(optionlist_cpy); - gras_assert1(FALSE, - "Value of option %s not valid. Should be an host (machine:port)", + xbt_assert1(FALSE, + "Value of option %s not valid. Should be an host (machine:port)", name); } @@ -656,13 +654,16 @@ gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { i=strtol(val, &val, 0); if (val==NULL) { free(optionlist_cpy); - gras_assert1(FALSE, - "Value of option %s not valid. Should be an host (machine:port)", + xbt_assert1(FALSE, + "Value of option %s not valid. Should be an host (machine:port)", name); } - TRYCLEAN(gras_cfg_set_host(cfg,name,str,i), - free(optionlist_cpy)); + errcode = xbt_cfg_set_host(cfg,name,str,i); + if (errcode != no_error) { + free(optionlist_cpy); + return errcode; + } break; default: @@ -675,131 +676,177 @@ gras_cfg_set_parse(gras_cfg_t *cfg, const char *options) { return no_error; } -/** - * gras_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 */ -gras_error_t -gras_cfg_set_int(gras_cfg_t *cfg,const char*name, int val) { - gras_cfgelm_t *cell; - gras_error_t errcode; - - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_int,&cell)); - - if (cell->max > 1) { - return gras_dynar_push(cell->content,&val); +xbt_error_t +xbt_cfg_set_int(xbt_cfg_t cfg,const char*name, int val) { + xbt_cfgelm_t variable; + xbt_error_t errcode; + + VERB2("Configuration setting: %s=%d",name,val); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&variable)); + + 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 { - return gras_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + RAISE3(mismatch_error, + "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); } + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); + return no_error; } -/** - * gras_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 */ -gras_error_t -gras_cfg_set_double(gras_cfg_t *cfg,const char*name, double val) { - gras_cfgelm_t *cell; - gras_error_t errcode; +xbt_error_t +xbt_cfg_set_double(xbt_cfg_t cfg,const char*name, double val) { + xbt_cfgelm_t variable; + xbt_error_t errcode; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_double,&cell)); + VERB2("Configuration setting: %s=%f",name,val); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&variable)); - if (cell->max > 1) { - return gras_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 { - return gras_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + RAISE3(mismatch_error, + "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); } + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); + return no_error; } -/** - * gras_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. */ -gras_error_t -gras_cfg_set_string(gras_cfg_t *cfg,const char*name, const char*val) { - gras_cfgelm_t *cell; - gras_error_t errcode; +xbt_error_t +xbt_cfg_set_string(xbt_cfg_t cfg,const char*name, const char*val) { + xbt_cfgelm_t variable; + xbt_error_t errcode; + char *newval = xbt_strdup(val); - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_string,&cell)); + VERB2("Configuration setting: %s=%s",name,val); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&variable)); - if (cell->max > 1) { - return gras_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,&newval); } else { - return gras_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + RAISE3(mismatch_error, + "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); } + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); + return no_error; } -/** - * gras_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) */ -gras_error_t -gras_cfg_set_host(gras_cfg_t *cfg,const char*name, +xbt_error_t +xbt_cfg_set_host(xbt_cfg_t cfg,const char*name, const char *host,int port) { - gras_cfgelm_t *cell; - gras_error_t errcode; - gras_host_t *val=malloc(sizeof(gras_host_t)); + xbt_cfgelm_t variable; + xbt_error_t errcode; + xbt_host_t *val=xbt_new(xbt_host_t,1); + + VERB3("Configuration setting: %s=%s:%d",name,host,port); - if (!val) - RAISE_MALLOC; - val->name = strdup(name); + val->name = xbt_strdup(name); val->port = port; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_host,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&variable)); - if (cell->max > 1) { - return gras_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 { - return gras_dynar_set(cell->content,0,&val); + if (variable->max && xbt_dynar_length(variable->content) == variable->max) + RAISE4(mismatch_error, + "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); } + + if (variable->cb_set) + (*variable->cb_set)(name, xbt_dynar_length(variable->content) -1); + return no_error; } /* ---- [ Removing ] ---- */ -/** - * gras_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 */ -gras_error_t gras_cfg_rm_int (gras_cfg_t *cfg,const char*name, int val) { +xbt_error_t xbt_cfg_rm_int(xbt_cfg_t cfg,const char*name, int val) { - gras_cfgelm_t *cell; + xbt_cfgelm_t variable; int cpt,seen; - gras_error_t errcode; + xbt_error_t errcode; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_int,&cell)); + if (xbt_dynar_length(variable->content) == variable->min) + RAISE3(mismatch_error, + "Cannot remove value %d from the config element %s since it's already at its minimal size (=%d)", + val,name,variable->min); + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&variable)); - gras_dynar_foreach(cell->content,cpt,seen) { + xbt_dynar_foreach(variable->content,cpt,seen) { if (seen == val) { - gras_dynar_cursor_rm(cell->content,&cpt); + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); return no_error; } } @@ -809,27 +856,30 @@ gras_error_t gras_cfg_rm_int (gras_cfg_t *cfg,const char*name, int val) { return mismatch_error; } -/** - * gras_cfg_rm_double: +/** @brief Remove the provided \e val double value from a variable * - * @cfg: the config set - * @name: the name of the cell - * @val: the value to 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 to be removed */ -gras_error_t gras_cfg_rm_double(gras_cfg_t *cfg,const char*name, double val) { - gras_cfgelm_t *cell; +xbt_error_t xbt_cfg_rm_double(xbt_cfg_t cfg,const char*name, double val) { + xbt_cfgelm_t variable; int cpt; double seen; - gras_error_t errcode; + xbt_error_t errcode; + + if (xbt_dynar_length(variable->content) == variable->min) + RAISE3(mismatch_error, + "Cannot remove value %f from the config element %s since it's already at its minimal size (=%d)", + val,name,variable->min); - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_double,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&variable)); - gras_dynar_foreach(cell->content,cpt,seen) { + xbt_dynar_foreach(variable->content,cpt,seen) { if (seen == val) { - gras_dynar_cursor_rm(cell->content,&cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); return no_error; } } @@ -839,27 +889,30 @@ gras_error_t gras_cfg_rm_double(gras_cfg_t *cfg,const char*name, double val) { return mismatch_error; } -/** - * gras_cfg_rm_string: - * - * @cfg: the config set - * @name: the name of the cell - * @val: the value of the string which will be removed +/** @brief Remove the provided \e val string 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 of the string which will be removed */ -gras_error_t -gras_cfg_rm_string(gras_cfg_t *cfg,const char*name, const char *val) { - gras_cfgelm_t *cell; +xbt_error_t +xbt_cfg_rm_string(xbt_cfg_t cfg,const char*name, const char *val) { + xbt_cfgelm_t variable; int cpt; char *seen; - gras_error_t errcode; - - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_string,&cell)); + xbt_error_t errcode; + + if (xbt_dynar_length(variable->content) == variable->min) + RAISE3(mismatch_error, + "Cannot remove value %s from the config element %s since it's already at its minimal size (=%d)", + name,val,variable->min); + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&variable)); - gras_dynar_foreach(cell->content,cpt,seen) { + xbt_dynar_foreach(variable->content,cpt,seen) { if (!strcpy(seen,val)) { - gras_dynar_cursor_rm(cell->content,&cpt); + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); return no_error; } } @@ -869,29 +922,32 @@ gras_cfg_rm_string(gras_cfg_t *cfg,const char*name, const char *val) { return mismatch_error; } -/** - * gras_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 */ -gras_error_t -gras_cfg_rm_host (gras_cfg_t *cfg,const char*name, const char *host,int port) { - gras_cfgelm_t *cell; +xbt_error_t +xbt_cfg_rm_host(xbt_cfg_t cfg,const char*name, const char *host,int port) { + xbt_cfgelm_t variable; int cpt; - gras_host_t *seen; - gras_error_t errcode; - - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_host,&cell)); + xbt_host_t *seen; + xbt_error_t errcode; + + if (xbt_dynar_length(variable->content) == variable->min) + RAISE4(mismatch_error, + "Cannot remove value %s:%d from the config element %s since it's already at its minimal size (=%d)", + host,port,name,variable->min); + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&variable)); - gras_dynar_foreach(cell->content,cpt,seen) { + xbt_dynar_foreach(variable->content,cpt,seen) { if (!strcpy(seen->name,host) && seen->port == port) { - gras_dynar_cursor_rm(cell->content,&cpt); + if (variable->cb_rm) (*variable->cb_rm)(name, cpt); + xbt_dynar_cursor_rm(variable->content,&cpt); return no_error; } } @@ -901,188 +957,283 @@ gras_cfg_rm_host (gras_cfg_t *cfg,const char*name, const char *host,int port) { return mismatch_error; } -/* rm everything */ +/** @brief Remove the \e pos th value from the provided variable */ -/** - * gras_cfg_empty: +xbt_error_t xbt_cfg_rm_at (xbt_cfg_t cfg, const char *name, int pos) { + + xbt_cfgelm_t variable; + int cpt; + xbt_host_t *seen; + xbt_error_t errcode; + + if (xbt_dynar_length(variable->content) == variable->min) + RAISE3(mismatch_error, + "Cannot remove %dth value from the config element %s since it's already at its minimal size (=%d)", + pos,name,variable->min); + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_any,&variable)); + + if (variable->cb_rm) (*variable->cb_rm)(name, pos); + xbt_dynar_remove_at(variable->content, pos, NULL); + + return mismatch_error; +} + +/** @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 */ -gras_error_t -gras_cfg_empty(gras_cfg_t *cfg,const char*name) { - gras_cfgelm_t *cell; +xbt_error_t +xbt_cfg_empty(xbt_cfg_t cfg,const char*name) { + xbt_cfgelm_t variable; - gras_error_t errcode; + xbt_error_t errcode; TRYCATCH(mismatch_error, - gras_dict_retrieve((gras_dict_t*)cfg,name,(void**)&cell)); + xbt_dict_get((xbt_dict_t)cfg,name,(void**)&variable)); if (errcode == mismatch_error) { ERROR1("Can't empty '%s' since this config element does not exist", name); return mismatch_error; } - if (cell) { - gras_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 ]---------------------------------------------------------*/ -/** - * gras_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 gras_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 */ -gras_error_t -gras_cfg_get_int (gras_cfg_t *cfg, +xbt_error_t +xbt_cfg_get_int (xbt_cfg_t cfg, const char *name, int *val) { - gras_cfgelm_t *cell; - gras_error_t errcode; + xbt_cfgelm_t variable; + xbt_error_t errcode; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_int,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&variable)); - if (gras_dynar_length(cell->content) > 1) { - WARN2("You asked for the first value of the config element '%s', but there is %d values\n", - name, gras_dynar_length(cell->content)); + 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(variable->content)); } - gras_dynar_get(cell->content, 0, (void*)val); + *val = xbt_dynar_get_as(variable->content, 0, int); return no_error; } -/** - * gras_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 gras_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 */ -gras_error_t -gras_cfg_get_double(gras_cfg_t *cfg, +xbt_error_t +xbt_cfg_get_double(xbt_cfg_t cfg, const char *name, double *val) { - gras_cfgelm_t *cell; - gras_error_t errcode; + xbt_cfgelm_t variable; + xbt_error_t errcode; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_double,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&variable)); - if (gras_dynar_length(cell->content) > 1) { - WARN2("You asked for the first value of the config element '%s', but there is %d values\n", - name, gras_dynar_length(cell->content)); + 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(variable->content)); } - gras_dynar_get(cell->content, 0, (void*)val); + *val = xbt_dynar_get_as(variable->content, 0, double); return no_error; } -/** - * gras_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 gras_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 */ -gras_error_t gras_cfg_get_string(gras_cfg_t *cfg, +xbt_error_t xbt_cfg_get_string(xbt_cfg_t cfg, const char *name, char **val) { - gras_cfgelm_t *cell; - gras_error_t errcode; + xbt_cfgelm_t variable; + xbt_error_t errcode; *val=NULL; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_string,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&variable)); - if (gras_dynar_length(cell->content) > 1) { - WARN2("You asked for the first value of the config element '%s', but there is %d values\n", - name, gras_dynar_length(cell->content)); + 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(variable->content)); } - gras_dynar_get(cell->content, 0, (void*)val); + *val = xbt_dynar_get_as(variable->content, 0, char *); return no_error; } -/** - * gras_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 gras_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 */ -gras_error_t gras_cfg_get_host (gras_cfg_t *cfg, +xbt_error_t xbt_cfg_get_host (xbt_cfg_t cfg, const char *name, char **host, int *port) { - gras_cfgelm_t *cell; - gras_error_t errcode; - gras_host_t *val; + xbt_cfgelm_t variable; + xbt_error_t errcode; + xbt_host_t *val; - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_host,&cell)); + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_host,&variable)); - if (gras_dynar_length(cell->content) > 1) { - WARN2("You asked for the first value of the config element '%s', but there is %d values\n", - name, gras_dynar_length(cell->content)); + 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(variable->content)); } - gras_dynar_get(cell->content, 0, (void*)val); + val = xbt_dynar_get_as(variable->content, 0, xbt_host_t*); *host=val->name; *port=val->port; return no_error; } -/** - * gras_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 */ -gras_error_t gras_cfg_get_dynar (gras_cfg_t *cfg, - const char *name, - gras_dynar_t **dynar) { - gras_cfgelm_t *cell; - gras_error_t errcode; - - TRY (gras_cfgelm_get(cfg,name,gras_cfgelm_host,&cell)); - *dynar = cell->content; +xbt_error_t xbt_cfg_get_dynar (xbt_cfg_t cfg, + const char *name, + xbt_dynar_t *dynar) { + xbt_cfgelm_t variable; + xbt_error_t errcode = xbt_dict_get((xbt_dict_t)cfg,name, + (void**)&variable); + + if (errcode == mismatch_error) { + ERROR1("No registered variable %s in this config set", + name); + return mismatch_error; + } + if (errcode != no_error) + return errcode; + + *dynar = variable->content; return no_error; } + +/** @brief Retrieve one of the integer value of a variable */ +xbt_error_t +xbt_cfg_get_int_at(xbt_cfg_t cfg, + const char *name, + int pos, + int *val) { + + xbt_cfgelm_t variable; + xbt_error_t errcode; + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&variable)); + *val = xbt_dynar_get_as(variable->content, pos, int); + return no_error; +} + +/** @brief Retrieve one of the double value of a variable */ +xbt_error_t +xbt_cfg_get_double_at(xbt_cfg_t cfg, + const char *name, + int pos, + double *val) { + + xbt_cfgelm_t variable; + xbt_error_t errcode; + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_double,&variable)); + *val = xbt_dynar_get_as(variable->content, pos, double); + return no_error; +} + + +/** @brief Retrieve one of the string value of a variable */ +xbt_error_t +xbt_cfg_get_string_at(xbt_cfg_t cfg, + const char *name, + int pos, + char **val) { + + xbt_cfgelm_t variable; + xbt_error_t errcode; + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_string,&variable)); + *val = xbt_dynar_get_as(variable->content, pos, char*); + return no_error; +} + +/** @brief Retrieve one of the host value of a variable */ +xbt_error_t +xbt_cfg_get_host_at(xbt_cfg_t cfg, + const char *name, + int pos, + char **host, + int *port) { + + xbt_cfgelm_t variable; + xbt_error_t errcode; + xbt_host_t *val; + + TRY (xbt_cfgelm_get(cfg,name,xbt_cfgelm_int,&variable)); + val = xbt_dynar_get_ptr(variable->content, pos); + *port = val->port; + *host = val->name; + return no_error; +}