X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/621567e097f236cdb81007637323ecc23ac26d45..d5c11dbc7ac52c97c04aee539dbf9e69a2a68f68:/src/xbt/dynar.c diff --git a/src/xbt/dynar.c b/src/xbt/dynar.c index 7c3bcc6eff..247a834d77 100644 --- a/src/xbt/dynar.c +++ b/src/xbt/dynar.c @@ -1,12 +1,11 @@ /* a generic DYNamic ARray implementation. */ -/* Copyright (c) 2004, 2005, 2006, 2007, 2008, 2009, 2010. The SimGrid Team. +/* Copyright (c) 2004-2013. The SimGrid Team. * 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. */ -#include "portable.h" /* SIZEOF_MAX */ #include "xbt/misc.h" #include "xbt/sysdep.h" #include "xbt/log.h" @@ -14,25 +13,8 @@ #include "xbt/dynar.h" #include -/* IMPLEMENTATION NOTE ON SYNCHRONIZATION: every functions which name is prefixed by _ - * assumes that the dynar is already locked if we have to. - * Other functions (public ones) check for this. - */ - XBT_LOG_NEW_DEFAULT_SUBCATEGORY(xbt_dyn, xbt, "Dynamic arrays"); -static XBT_INLINE void _dynar_lock(xbt_dynar_t dynar) -{ - if (dynar->mutex) - xbt_mutex_acquire(dynar->mutex); -} - -static XBT_INLINE void _dynar_unlock(xbt_dynar_t dynar) -{ - if (dynar->mutex) - xbt_mutex_release(dynar->mutex); -} - static XBT_INLINE void _sanity_check_dynar(xbt_dynar_t dynar) { xbt_assert(dynar, "dynar is NULL"); @@ -46,38 +28,26 @@ static XBT_INLINE void _sanity_check_idx(int idx) static XBT_INLINE void _check_inbound_idx(xbt_dynar_t dynar, int idx) { if (idx < 0 || idx >= dynar->used) { - _dynar_unlock(dynar); THROWF(bound_error, idx, "dynar is not that long. You asked %d, but it's only %lu long", (int) (idx), (unsigned long) dynar->used); } } -static XBT_INLINE void _check_sloppy_inbound_idx(xbt_dynar_t dynar, - int idx) -{ - if (idx > dynar->used) { - _dynar_unlock(dynar); - THROWF(bound_error, idx, - "dynar is not that long. You asked %d, but it's only %lu long (could have been equal to it)", - (int) (idx), (unsigned long) dynar->used); - } -} - static XBT_INLINE void _check_populated_dynar(xbt_dynar_t dynar) { if (dynar->used == 0) { - _dynar_unlock(dynar); THROWF(bound_error, 0, "dynar %p is empty", dynar); } } -static void _dynar_map(const xbt_dynar_t dynar, void_f_pvoid_t const op); - static XBT_INLINE - void _xbt_clear_mem(void *const ptr, const unsigned long length) +void _xbt_dynar_resize(xbt_dynar_t dynar, unsigned long new_size) { - memset(ptr, 0, length); + if (new_size != dynar->size) { + dynar->size = new_size; + dynar->data = xbt_realloc(dynar->data, new_size * dynar->elmsize); + } } static XBT_INLINE @@ -86,30 +56,10 @@ static XBT_INLINE const unsigned long old_size = dynar->size; if (nb > old_size) { - char *const old_data = (char *) dynar->data; - - const unsigned long elmsize = dynar->elmsize; - - const unsigned long used = dynar->used; - const unsigned long used_length = used * elmsize; - - const unsigned long new_size = - nb > (2 * (old_size + 1)) ? nb : (2 * (old_size + 1)); - const unsigned long new_length = new_size * elmsize; - char *const new_data = (char *) xbt_malloc0(elmsize * new_size); - - XBT_DEBUG("expand %p from %lu to %lu elements", (void *) dynar, - (unsigned long) old_size, nb); - - if (old_data) { - memcpy(new_data, old_data, used_length); - free(old_data); - } - - _xbt_clear_mem(new_data + used_length, new_length - used_length); - - dynar->size = new_size; - dynar->data = new_data; + const unsigned long expand = 2 * (old_size + 1); + _xbt_dynar_resize(dynar, (nb > expand ? nb : expand)); + XBT_DEBUG("expand %p from %lu to %lu elements", + dynar, old_size, dynar->size); } } @@ -143,44 +93,6 @@ _xbt_dynar_put_elm(const xbt_dynar_t dynar, memcpy(elm, src, elmsize); } -static XBT_INLINE - void -_xbt_dynar_remove_at(xbt_dynar_t const dynar, - const unsigned long idx, void *const object) -{ - - unsigned long nb_shift; - unsigned long offset; - - _sanity_check_dynar(dynar); - _check_inbound_idx(dynar, idx); - - if (object) { - _xbt_dynar_get_elm(object, dynar, idx); - } else if (dynar->free_f) { - if (dynar->elmsize <= SIZEOF_MAX) { - char elm[SIZEOF_MAX]; - _xbt_dynar_get_elm(elm, dynar, idx); - (*dynar->free_f) (elm); - } else { - char *elm = malloc(dynar->elmsize); - _xbt_dynar_get_elm(elm, dynar, idx); - (*dynar->free_f) (elm); - free(elm); - } - } - - nb_shift = dynar->used - 1 - idx; - - if (nb_shift) { - offset = nb_shift * dynar->elmsize; - memmove(_xbt_dynar_elm(dynar, idx), _xbt_dynar_elm(dynar, idx + 1), - offset); - } - - dynar->used--; -} - void xbt_dynar_dump(xbt_dynar_t dynar) { XBT_INFO("Dynar dump: size=%lu; used=%lu; elmsize=%lu; data=%p; free_f=%p", @@ -208,25 +120,10 @@ xbt_dynar_new(const unsigned long elmsize, void_f_pvoid_t const free_f) dynar->elmsize = elmsize; dynar->data = NULL; dynar->free_f = free_f; - dynar->mutex = NULL; return dynar; } -/** @brief Creates a synchronized dynar. - * - * Just like #xbt_dynar_new, but each access to the structure will be protected by a mutex - * - */ -xbt_dynar_t -xbt_dynar_new_sync(const unsigned long elmsize, - void_f_pvoid_t const free_f) -{ - xbt_dynar_t res = xbt_dynar_new(elmsize, free_f); - res->mutex = xbt_mutex_init(); - return res; -} - /** @brief Destructor of the structure not touching to the content * * \param dynar poor victim @@ -237,18 +134,9 @@ xbt_dynar_new_sync(const unsigned long elmsize, void xbt_dynar_free_container(xbt_dynar_t * dynar) { if (dynar && *dynar) { - - if ((*dynar)->data) { - _xbt_clear_mem((*dynar)->data, (*dynar)->size); - free((*dynar)->data); - } - - if ((*dynar)->mutex) - xbt_mutex_destroy((*dynar)->mutex); - - _xbt_clear_mem(*dynar, sizeof(s_xbt_dynar_t)); - - free(*dynar); + xbt_dynar_t d = *dynar; + free(d->data); + free(d); *dynar = NULL; } } @@ -259,25 +147,35 @@ void xbt_dynar_free_container(xbt_dynar_t * dynar) */ XBT_INLINE void xbt_dynar_reset(xbt_dynar_t const dynar) { - _dynar_lock(dynar); - _sanity_check_dynar(dynar); - XBT_DEBUG("Reset the dynar %p", (void *) dynar); + XBT_CDEBUG(xbt_dyn, "Reset the dynar %p", (void *) dynar); if (dynar->free_f) { - _dynar_map(dynar, dynar->free_f); + xbt_dynar_map(dynar, dynar->free_f); } - /* - if (dynar->data) - free(dynar->data); - - dynar->size = 0; - */ dynar->used = 0; +} + +/** @brief Merge dynar d2 into d1 + * + * \param d1 dynar to keep + * \param d2 dynar to merge into d1. This dynar is free at end. + */ +void xbt_dynar_merge(xbt_dynar_t *d1, xbt_dynar_t *d2) +{ + if((*d1)->elmsize != (*d2)->elmsize) + xbt_die("Element size must are not equal"); + + const unsigned long elmsize = (*d1)->elmsize; - _dynar_unlock(dynar); + void *ptr = _xbt_dynar_elm((*d2), 0); + _xbt_dynar_resize(*d1, (*d1)->size + (*d2)->size); + void *elm = _xbt_dynar_elm((*d1), (*d1)->used); - /* dynar->data = NULL; */ + memcpy(elm, ptr, ((*d2)->size)*elmsize); + (*d1)->used += (*d2)->used; + (*d2)->used = 0; + xbt_dynar_free(d2); } /** @@ -297,16 +195,7 @@ XBT_INLINE void xbt_dynar_reset(xbt_dynar_t const dynar) */ void xbt_dynar_shrink(xbt_dynar_t dynar, int empty_slots_wanted) { - unsigned long size_wanted; - - _dynar_lock(dynar); - - size_wanted = dynar->used + empty_slots_wanted; - if (size_wanted != dynar->size) { - dynar->size = size_wanted; - dynar->data = xbt_realloc(dynar->data, dynar->elmsize * dynar->size); - } - _dynar_unlock(dynar); + _xbt_dynar_resize(dynar, dynar->used + empty_slots_wanted); } /** @brief Destructor @@ -340,7 +229,7 @@ XBT_INLINE unsigned long xbt_dynar_length(const xbt_dynar_t dynar) return (dynar ? (unsigned long) dynar->used : (unsigned long) 0); } -/**@brief check if a dynar is empty + /**@brief check if a dynar is empty * *\param dynar the dynat we want to check */ @@ -360,12 +249,10 @@ XBT_INLINE void xbt_dynar_get_cpy(const xbt_dynar_t dynar, const unsigned long idx, void *const dst) { - _dynar_lock(dynar); _sanity_check_dynar(dynar); _check_inbound_idx(dynar, idx); _xbt_dynar_get_elm(dst, dynar, idx); - _dynar_unlock(dynar); } /** @brief Retrieve a pointer to the Nth element of a dynar. @@ -382,54 +269,27 @@ XBT_INLINE void *xbt_dynar_get_ptr(const xbt_dynar_t dynar, { void *res; - _dynar_lock(dynar); _sanity_check_dynar(dynar); _check_inbound_idx(dynar, idx); res = _xbt_dynar_elm(dynar, idx); - _dynar_unlock(dynar); return res; } XBT_INLINE void *xbt_dynar_set_at_ptr(const xbt_dynar_t dynar, const unsigned long idx) { - - void *res; - _dynar_lock(dynar); - _sanity_check_dynar(dynar); - - _xbt_dynar_expand(dynar, idx + 1); - - if (idx >= dynar->used) { - _xbt_clear_mem(((char * const)dynar->data) + dynar->used * dynar->elmsize, - (idx + 1 - dynar->used)*dynar->elmsize); - dynar->used = idx + 1; - } - - _dynar_unlock(dynar); - - res = _xbt_dynar_elm(dynar, idx); - - return res; -} - -static void XBT_INLINE /* not synchronized */ -_xbt_dynar_set(xbt_dynar_t dynar, - const unsigned long idx, const void *const src) -{ - _sanity_check_dynar(dynar); - _xbt_dynar_expand(dynar, idx + 1); - if (idx >= dynar->used) { - _xbt_clear_mem(((char * const)dynar->data) + dynar->used * dynar->elmsize, - (idx + 1 - dynar->used)*dynar->elmsize); + _xbt_dynar_expand(dynar, idx + 1); + if (idx > dynar->used) { + memset(_xbt_dynar_elm(dynar, dynar->used), 0, + (idx - dynar->used) * dynar->elmsize); + } dynar->used = idx + 1; } - - _xbt_dynar_put_elm(dynar, idx, src); + return _xbt_dynar_elm(dynar, idx); } /** @brief Set the Nth element of a dynar (expanded if needed). Previous value at this position is NOT freed @@ -444,9 +304,7 @@ XBT_INLINE void xbt_dynar_set(xbt_dynar_t dynar, const int idx, const void *const src) { - _dynar_lock(dynar); - _xbt_dynar_set(dynar, idx, src); - _dynar_unlock(dynar); + memcpy(xbt_dynar_set_at_ptr(dynar, idx), src, dynar->elmsize); } /** @brief Set the Nth element of a dynar (expanded if needed). Previous value is freed @@ -463,21 +321,23 @@ void xbt_dynar_replace(xbt_dynar_t dynar, const unsigned long idx, const void *const object) { - _dynar_lock(dynar); _sanity_check_dynar(dynar); if (idx < dynar->used && dynar->free_f) { void *const old_object = _xbt_dynar_elm(dynar, idx); - (*(dynar->free_f)) (old_object); + dynar->free_f(old_object); } - _xbt_dynar_set(dynar, idx, object); - _dynar_unlock(dynar); + xbt_dynar_set(dynar, idx, object); } -static XBT_INLINE void *_xbt_dynar_insert_at_ptr(xbt_dynar_t const dynar, - const unsigned long idx) +/** @brief Make room for a new element, and return a pointer to it + * + * You can then use regular affectation to set its value instead of relying + * on the slow memcpy. This is what xbt_dynar_insert_at_as() does. + */ +void *xbt_dynar_insert_at_ptr(xbt_dynar_t const dynar, const int idx) { void *res; unsigned long old_used; @@ -504,21 +364,6 @@ static XBT_INLINE void *_xbt_dynar_insert_at_ptr(xbt_dynar_t const dynar, return res; } -/** @brief Make room for a new element, and return a pointer to it - * - * You can then use regular affectation to set its value instead of relying - * on the slow memcpy. This is what xbt_dynar_insert_at_as() does. - */ -void *xbt_dynar_insert_at_ptr(xbt_dynar_t const dynar, const int idx) -{ - void *res; - - _dynar_lock(dynar); - res = _xbt_dynar_insert_at_ptr(dynar, idx); - _dynar_unlock(dynar); - return res; -} - /** @brief Set the Nth dynar's element, expanding the dynar and sliding the previous values to the right * * Set the Nth element of a dynar, expanding the dynar if needed, and @@ -530,10 +375,8 @@ xbt_dynar_insert_at(xbt_dynar_t const dynar, const int idx, const void *const src) { - _dynar_lock(dynar); /* checks done in xbt_dynar_insert_at_ptr */ - memcpy(_xbt_dynar_insert_at_ptr(dynar, idx), src, dynar->elmsize); - _dynar_unlock(dynar); + memcpy(xbt_dynar_insert_at_ptr(dynar, idx), src, dynar->elmsize); } /** @brief Remove the Nth dynar's element, sliding the previous values to the left @@ -549,33 +392,127 @@ void xbt_dynar_remove_at(xbt_dynar_t const dynar, const int idx, void *const object) { + unsigned long nb_shift; + unsigned long offset; + + _sanity_check_dynar(dynar); + _check_inbound_idx(dynar, idx); + + if (object) { + _xbt_dynar_get_elm(object, dynar, idx); + } else if (dynar->free_f) { + dynar->free_f(_xbt_dynar_elm(dynar, idx)); + } + + nb_shift = dynar->used - 1 - idx; + + if (nb_shift) { + offset = nb_shift * dynar->elmsize; + memmove(_xbt_dynar_elm(dynar, idx), _xbt_dynar_elm(dynar, idx + 1), + offset); + } + + dynar->used--; +} + +/** @brief Remove a slice of the dynar, sliding the rest of the values to the left + * + * This function removes an n-sized slice that starts at element idx. It is equivalent + * to xbt_dynar_remove_at with a NULL object argument if n equals to 1. + * + * Each of the removed elements is freed using the free_f function passed at dynar + * creation. + */ +void +xbt_dynar_remove_n_at(xbt_dynar_t const dynar, + const unsigned int n, const int idx) +{ + unsigned long nb_shift; + unsigned long offset; + unsigned long cur; - _dynar_lock(dynar); - _xbt_dynar_remove_at(dynar, idx, object); - _dynar_unlock(dynar); + if (!n) return; + + _sanity_check_dynar(dynar); + _check_inbound_idx(dynar, idx); + _check_inbound_idx(dynar, idx + n - 1); + + if (dynar->free_f) { + for (cur = idx; cur < idx + n; cur++) { + dynar->free_f(_xbt_dynar_elm(dynar, cur)); + } + } + + nb_shift = dynar->used - n - idx; + + if (nb_shift) { + offset = nb_shift * dynar->elmsize; + memmove(_xbt_dynar_elm(dynar, idx), _xbt_dynar_elm(dynar, idx + n), + offset); + } + + dynar->used -= n; } /** @brief Returns the position of the element in the dynar * - * Raises not_found_error if not found. + * Beware that if your dynar contains pointed values (such as strings) instead + * of scalar, this function compares the pointer value, not what's pointed. The only + * solution to search for a pointed value is then to write the foreach loop yourself: + * \code + * signed int position = -1; + * xbt_dynar_foreach(dynar, iter, elem) { + * if (!memcmp(elem, searched_element, sizeof(*elem))) { + * position = iter; + * break; + * } + * } + * \endcode + * + * Raises not_found_error if not found. If you have less than 2 millions elements, + * you probably want to use #xbt_dynar_search_or_negative() instead, so that you + * don't have to TRY/CATCH on element not found. */ unsigned int xbt_dynar_search(xbt_dynar_t const dynar, void *const elem) { unsigned long it; - _dynar_lock(dynar); for (it = 0; it < dynar->used; it++) if (!memcmp(_xbt_dynar_elm(dynar, it), elem, dynar->elmsize)) { - _dynar_unlock(dynar); return it; } - _dynar_unlock(dynar); THROWF(not_found_error, 0, "Element %p not part of dynar %p", elem, dynar); } -/** @brief Returns a boolean indicating whether the element is part of the dynar */ +/** @brief Returns the position of the element in the dynar (or -1 if not found) + * + * Beware that if your dynar contains pointed values (such as + * strings) instead of scalar, this function is probably not what you + * want. Check the documentation of xbt_dynar_search() for more info. + * + * Note that usually, the dynar indices are unsigned integers. If you have more + * than 2 million elements in your dynar, this very function will not work (but the other will). + */ +signed int xbt_dynar_search_or_negative(xbt_dynar_t const dynar, void *const elem) +{ + unsigned long it; + + for (it = 0; it < dynar->used; it++) + if (!memcmp(_xbt_dynar_elm(dynar, it), elem, dynar->elmsize)) { + return it; + } + + return -1; +} + +/** @brief Returns a boolean indicating whether the element is part of the dynar + * + * Beware that if your dynar contains pointed values (such as + * strings) instead of scalar, this function is probably not what you + * want. Check the documentation of xbt_dynar_search() for more info. + */ int xbt_dynar_member(xbt_dynar_t const dynar, void *const elem) { @@ -583,7 +520,8 @@ int xbt_dynar_member(xbt_dynar_t const dynar, void *const elem) TRY { xbt_dynar_search(dynar, elem); - } CATCH(e) { + } + CATCH(e) { if (e.category == not_found_error) { xbt_ex_free(e); return 0; @@ -600,26 +538,15 @@ int xbt_dynar_member(xbt_dynar_t const dynar, void *const elem) */ XBT_INLINE void *xbt_dynar_push_ptr(xbt_dynar_t const dynar) { - void *res; - - /* we have to inline xbt_dynar_insert_at_ptr here to make sure that - dynar->used don't change between reading it and getting the lock - within xbt_dynar_insert_at_ptr */ - _dynar_lock(dynar); - res = _xbt_dynar_insert_at_ptr(dynar, dynar->used); - _dynar_unlock(dynar); - return res; + return xbt_dynar_insert_at_ptr(dynar, dynar->used); } /** @brief Add an element at the end of the dynar */ XBT_INLINE void xbt_dynar_push(xbt_dynar_t const dynar, const void *const src) { - _dynar_lock(dynar); /* checks done in xbt_dynar_insert_at_ptr */ - memcpy(_xbt_dynar_insert_at_ptr(dynar, dynar->used), src, - dynar->elmsize); - _dynar_unlock(dynar); + memcpy(xbt_dynar_insert_at_ptr(dynar, dynar->used), src, dynar->elmsize); } /** @brief Mark the last dynar's element as unused and return a pointer to it. @@ -629,15 +556,10 @@ XBT_INLINE void xbt_dynar_push(xbt_dynar_t const dynar, */ XBT_INLINE void *xbt_dynar_pop_ptr(xbt_dynar_t const dynar) { - void *res; - - _dynar_lock(dynar); _check_populated_dynar(dynar); - XBT_DEBUG("Pop %p", (void *) dynar); + XBT_CDEBUG(xbt_dyn, "Pop %p", (void *) dynar); dynar->used--; - res = _xbt_dynar_elm(dynar, dynar->used); - _dynar_unlock(dynar); - return res; + return _xbt_dynar_elm(dynar, dynar->used); } /** @brief Get and remove the last element of the dynar */ @@ -645,10 +567,8 @@ XBT_INLINE void xbt_dynar_pop(xbt_dynar_t const dynar, void *const dst) { /* sanity checks done by remove_at */ - XBT_DEBUG("Pop %p", (void *) dynar); - _dynar_lock(dynar); - _xbt_dynar_remove_at(dynar, dynar->used - 1, dst); - _dynar_unlock(dynar); + XBT_CDEBUG(xbt_dyn, "Pop %p", (void *) dynar); + xbt_dynar_remove_at(dynar, dynar->used - 1, dst); } /** @brief Add an element at the begining of the dynar. @@ -674,38 +594,25 @@ XBT_INLINE void xbt_dynar_shift(xbt_dynar_t const dynar, void *const dst) xbt_dynar_remove_at(dynar, 0, dst); } -static void _dynar_map(const xbt_dynar_t dynar, void_f_pvoid_t const op) -{ - char *const data = (char *) dynar->data; - const unsigned long elmsize = dynar->elmsize; - const unsigned long used = dynar->used; - unsigned long i; - - for (i = 0; i < used; i++) { - char* elm = (char*) data + i * elmsize; - (*op) (elm); - } -} - /** @brief Apply a function to each member of a dynar * * The mapped function may change the value of the element itself, * but should not mess with the structure of the dynar. - * - * If the dynar is synchronized, it is locked during the whole map - * operation, so make sure your function don't call any function - * from xbt_dynar_* on it, or you'll get a deadlock. */ XBT_INLINE void xbt_dynar_map(const xbt_dynar_t dynar, void_f_pvoid_t const op) { + char *const data = (char *) dynar->data; + const unsigned long elmsize = dynar->elmsize; + const unsigned long used = dynar->used; + unsigned long i; _sanity_check_dynar(dynar); - _dynar_lock(dynar); - - _dynar_map(dynar, op); - _dynar_unlock(dynar); + for (i = 0; i < used; i++) { + char* elm = (char*) data + i * elmsize; + op(elm); + } } @@ -717,18 +624,7 @@ XBT_INLINE void xbt_dynar_cursor_rm(xbt_dynar_t dynar, unsigned int *const cursor) { - _xbt_dynar_remove_at(dynar, (*cursor)--, NULL); -} - -/** @brief Unlocks a synchronized dynar when you want to break the traversal - * - * This function must be used if you break the - * xbt_dynar_foreach loop, but shouldn't be called at the end of a - * regular traversal reaching the end of the elements - */ -XBT_INLINE void xbt_dynar_cursor_unlock(xbt_dynar_t dynar) -{ - _dynar_unlock(dynar); + xbt_dynar_remove_at(dynar, (*cursor)--, NULL); } /** @brief Sorts a dynar according to the function compar_fn @@ -742,73 +638,109 @@ XBT_INLINE void xbt_dynar_cursor_unlock(xbt_dynar_t dynar) XBT_INLINE void xbt_dynar_sort(xbt_dynar_t dynar, int_f_cpvoid_cpvoid_t compar_fn) { - - _dynar_lock(dynar); - -#ifdef HAVE_MERGESORT - mergesort(dynar->data, dynar->used, dynar->elmsize, compar_fn); -#else qsort(dynar->data, dynar->used, dynar->elmsize, compar_fn); -#endif - _dynar_unlock(dynar); } -/** @brief Transform a dynar into a NULL terminated array +/** @brief Sorts a dynar according to their color assuming elements can have only three colors. + * Since there are only three colors, it is linear and much faster than a classical sort. + * See for example http://en.wikipedia.org/wiki/Dutch_national_flag_problem * + * \param dynar the dynar to sort + * \param color the color function of type (int (compar_fn*) (void*) (void*)). The return value of color is assumed to be 0, 1, or 2. + * + * At the end of the call, elements with color 0 are at the beginning of the dynar, elements with color 2 are at the end and elements with color 1 are in the middle. + * + * Remark: if the elements stored in the dynar are structures, the color + * function has to retrieve the field to sort first. + */ +XBT_PUBLIC(void) xbt_dynar_three_way_partition(xbt_dynar_t const dynar, + int_f_pvoid_t color) +{ + unsigned long int i; + unsigned long int p = -1; + unsigned long int q = dynar->used; + const unsigned long elmsize = dynar->elmsize; + void *tmp = xbt_malloc(elmsize); + void *elm; + + for (i = 0; i < q;) { + void *elmi = _xbt_dynar_elm(dynar, i); + int colori = color(elmi); + + if (colori == 1) { + ++i; + } else { + if (colori == 0) { + elm = _xbt_dynar_elm(dynar, ++p); + ++i; + } else { /* colori == 2 */ + elm = _xbt_dynar_elm(dynar, --q); + } + if (elm != elmi) { + memcpy(tmp, elm, elmsize); + memcpy(elm, elmi, elmsize); + memcpy(elmi, tmp, elmsize); + } + } + } + xbt_free(tmp); +} + +/** @brief Transform a dynar into a NULL terminated array. + * The dynar won't be usable afterwards. * \param dynar the dynar to transform */ XBT_INLINE void * xbt_dynar_to_array (xbt_dynar_t dynar) { - void * res; - void * last = xbt_new0(char,dynar->elmsize); - xbt_dynar_push(dynar, last); - free(last); - res = dynar->data; - free(dynar); - return res; + void *res; + xbt_dynar_shrink(dynar, 1); + memset(xbt_dynar_push_ptr(dynar), 0, dynar->elmsize); + res = dynar->data; + free(dynar); + return res; } /* * Return 0 if d1 and d2 are equal and 1 if not equal */ -XBT_INLINE int xbt_dynar_compare(xbt_dynar_t d1, xbt_dynar_t d2, - int(*compar)(const void *, const void *)) +int xbt_dynar_compare(xbt_dynar_t d1, xbt_dynar_t d2, + int(*compar)(const void *, const void *)) { - int i ; - int size; - if((!d1) && (!d2)) return 0; - if((!d1) || (!d2)) - { - XBT_DEBUG("NULL dynar d1=%p d2=%p",d1,d2); - xbt_dynar_free(&d2); - return 1; - } - if((d1->elmsize)!=(d2->elmsize)) - { - XBT_DEBUG("Size of elmsize d1=%ld d2=%ld",d1->elmsize,d2->elmsize); - xbt_dynar_free(&d2); - return 1; // xbt_die - } - if(xbt_dynar_length(d1) != xbt_dynar_length(d2)) - { - XBT_DEBUG("Size of dynar d1=%ld d2=%ld",xbt_dynar_length(d1),xbt_dynar_length(d2)); - xbt_dynar_free(&d2); - return 1; - } - - size = xbt_dynar_length(d1); - for(i=0;ielmsize)!=(d2->elmsize)) + { + XBT_DEBUG("Size of elmsize d1=%lu d2=%lu",d1->elmsize,d2->elmsize); + xbt_dynar_free(&d2); + return 1; // xbt_die + } + if(xbt_dynar_length(d1) != xbt_dynar_length(d2)) + { + XBT_DEBUG("Size of dynar d1=%lu d2=%lu",xbt_dynar_length(d1),xbt_dynar_length(d2)); + xbt_dynar_free(&d2); + return 1; + } + + size = xbt_dynar_length(d1); + for(i=0;i