--- libaitcfg/src/queue.c 2012/04/04 13:11:49 1.7 +++ libaitcfg/src/queue.c 2014/01/29 23:48:34 1.13 @@ -3,7 +3,7 @@ * by Michael Pounov * * $Author: misho $ -* $Id: queue.c,v 1.7 2012/04/04 13:11:49 misho Exp $ +* $Id: queue.c,v 1.13 2014/01/29 23:48:34 misho Exp $ * ************************************************************************** The ELWIX and AITNET software is distributed under the following @@ -12,7 +12,7 @@ terms: All of the documentation and software included in the ELWIX and AITNET Releases is copyrighted by ELWIX - Sofia/Bulgaria -Copyright 2004, 2005, 2006, 2007, 2008, 2009, 2010, 2011, 2012 +Copyright 2004 - 2014 by Michael Pounov . All rights reserved. Redistribution and use in source and binary forms, with or without @@ -49,7 +49,7 @@ SUCH DAMAGE. static inline struct tagCfg * _selectAttribute(cfg_root_t * __restrict cfg, const char *csSec, const char *csAttr) { - struct tagCfg fav; + struct tagCfg fav, *c; if (!cfg) return NULL; @@ -58,15 +58,23 @@ _selectAttribute(cfg_root_t * __restrict cfg, const ch if (csSec && *csSec) AIT_KEY(&fav.cfg_sec) = crcFletcher16((u_short*) csSec, - io_align(strlen(csSec), 1) / 2); + E_ALIGN(strlen(csSec), 2) / 2); if (csAttr) AIT_KEY(&fav.cfg_attr) = crcFletcher16((u_short*) csAttr, - io_align(strlen(csAttr), 1) / 2); + E_ALIGN(strlen(csAttr), 2) / 2); if (!csAttr) return RB_NFIND(tagRC, cfg, &fav); - else - return RB_FIND(tagRC, cfg, &fav); + else { + c = RB_FIND(tagRC, cfg, &fav); + if (!c) + return NULL; /* not found */ + do { + if (!strcmp(AIT_GET_STR(&c->cfg_attr), csAttr)) + return c; /* FOUND! */ + } while (RB_NEXT(tagRC, cfg, c) && !cfg_tree_cmp(c, &fav)); + return NULL; /* not found */ + } } /* --------------------------------------------------------------- */ @@ -79,7 +87,7 @@ _selectAttribute(cfg_root_t * __restrict cfg, const ch * @csAttr = Config attribute //{csAttr} = ... * return: 0 not found item, -1 error or >0 position in list */ -inline int +int cfg_findAttribute(cfg_root_t * __restrict cfg, const char *csSec, const char *csAttr) { struct tagCfg *av, fav; @@ -93,10 +101,10 @@ cfg_findAttribute(cfg_root_t * __restrict cfg, const c if (csSec && *csSec) AIT_KEY(&fav.cfg_sec) = crcFletcher16((u_short*) csSec, - io_align(strlen(csSec), 1) / 2); + E_ALIGN(strlen(csSec), 2) / 2); if (csAttr) AIT_KEY(&fav.cfg_attr) = crcFletcher16((u_short*) csAttr, - io_align(strlen(csAttr), 1) / 2); + E_ALIGN(strlen(csAttr), 2) / 2); SLIST_FOREACH(av, cfg, cfg_next) { ++cx; @@ -135,7 +143,7 @@ cfg_unsetAttribute(cfg_root_t * __restrict cfg, const AIT_FREE_VAL(&av->cfg_val); AIT_FREE_VAL(&av->cfg_attr); AIT_FREE_VAL(&av->cfg_sec); - free(av); + e_free(av); return 1; } @@ -161,7 +169,7 @@ cfg_setAttribute(cfg_root_t * __restrict cfg, const ch /* adding new element */ section = _selectAttribute(cfg, csSec, NULL); - av = malloc(sizeof(struct tagCfg)); + av = e_malloc(sizeof(struct tagCfg)); if (!av) { LOGERR; return -1; @@ -179,12 +187,12 @@ cfg_setAttribute(cfg_root_t * __restrict cfg, const ch if (csSec && *csSec) { AIT_SET_STR(&av->cfg_sec, csSec); AIT_KEY(&av->cfg_sec) = crcFletcher16(AIT_GET_LIKE(&av->cfg_sec, u_short*), - io_align(AIT_LEN(&av->cfg_sec) - 1, 1) / 2); + E_ALIGN(AIT_LEN(&av->cfg_sec) - 1, 2) / 2); } AIT_SET_STR(&av->cfg_val, csVal ? csVal : ""); AIT_SET_STR(&av->cfg_attr, csAttr); AIT_KEY(&av->cfg_attr) = crcFletcher16(AIT_GET_LIKE(&av->cfg_attr, u_short*), - io_align(AIT_LEN(&av->cfg_attr) - 1, 1) / 2); + E_ALIGN(AIT_LEN(&av->cfg_attr) - 1, 2) / 2); CFG_RC_LOCK(cfg); RB_INSERT(tagRC, cfg, av); @@ -192,7 +200,8 @@ cfg_setAttribute(cfg_root_t * __restrict cfg, const ch return 2; } - if (csVal && strcmp((char*) csVal, (char*) AIT_GET_STR(&av->cfg_val))) { + if (csVal && AIT_ADDR(&av->cfg_val) && + strcmp((char*) csVal, (char*) AIT_GET_STR(&av->cfg_val))) { /* Update element */ AIT_FREE_VAL(&av->cfg_val); AIT_SET_STR(&av->cfg_val, csVal); @@ -211,7 +220,7 @@ cfg_setAttribute(cfg_root_t * __restrict cfg, const ch * @csAttr = Config attribute //{csAttr} = ..., if NULL unset as *any* attribute * return: NULL item not found or null parameters, !=NULL value const string */ -inline const char * +const char * cfg_getAttribute(cfg_root_t * __restrict cfg, const char *csSec, const char *csAttr) { struct tagCfg *av; @@ -248,22 +257,26 @@ cfg_loadAttribute(cfg_root_t * __restrict cfg, const c return -1; } + AIT_INIT_VAL(val); av = _selectAttribute(cfg, csSec, csAttr); if (!av) { /* not found item */ if (csDefValue) { AIT_SET_STR(val, csDefValue); ret = AIT_LEN(val); - } + } else + AIT_INIT_VAL(val); return ret; } - if (AIT_ISEMPTY(&av->cfg_val) || !*AIT_GET_LIKE(&av->cfg_val, char*)) { + if (AIT_ISEMPTY(&av->cfg_val) || !AIT_ADDR(&av->cfg_val) || + !*AIT_GET_LIKE(&av->cfg_val, char*)) { /* empty value */ if (csDefValue) { AIT_SET_STR(val, csDefValue); ret = AIT_LEN(val); - } + } else + AIT_INIT_VAL(val); } else { /* copy value */ AIT_SET_STR(val, AIT_GET_STR(&av->cfg_val));