X-Git-Url: http://info.iut-bm.univ-fcomte.fr/pub/gitweb/simgrid.git/blobdiff_plain/e99350b90a8c0e400db2f43e3093b70afdbd2aac..d9cad55dcc546b3936c278258e7f463ac70066df:/src/xbt/fifo.c diff --git a/src/xbt/fifo.c b/src/xbt/fifo.c index de5e0b7433..305abed358 100644 --- a/src/xbt/fifo.c +++ b/src/xbt/fifo.c @@ -13,7 +13,7 @@ XBT_LOG_NEW_DEFAULT_SUBCATEGORY(xbt_fifo, xbt, "FIFO"); static void *fifo_item_mallocator_new_f(void); -static void fifo_item_mallocator_free_f(void *item); +#define fifo_item_mallocator_free_f xbt_free_f static void fifo_item_mallocator_reset_f(void *item); static xbt_mallocator_t item_mallocator = NULL; @@ -143,7 +143,7 @@ void *xbt_fifo_shift(xbt_fifo_t l) */ void xbt_fifo_push_item(xbt_fifo_t l, xbt_fifo_item_t new) { - xbt_assert0((new->next == NULL) && (new->prev == NULL), "Invalid item!"); + xbt_assert((new->next == NULL) && (new->prev == NULL), "Invalid item!"); (l->count)++; if (l->head == NULL) { l->head = new; @@ -191,7 +191,7 @@ xbt_fifo_item_t xbt_fifo_pop_item(xbt_fifo_t l) */ void xbt_fifo_unshift_item(xbt_fifo_t l, xbt_fifo_item_t new) { - xbt_assert0((new->next == NULL) && (new->prev == NULL), "Invalid item!"); + xbt_assert((new->next == NULL) && (new->prev == NULL), "Invalid item!"); (l->count)++; if (l->head == NULL) { l->head = new; @@ -293,7 +293,7 @@ int xbt_fifo_remove_all(xbt_fifo_t l, void *t) void xbt_fifo_remove_item(xbt_fifo_t l, xbt_fifo_item_t current) { if (l->head == l->tail) { /* special case */ - xbt_assert0((current == l->head), "This item is not in the list!"); + xbt_assert((current == l->head), "This item is not in the list!"); l->head = NULL; l->tail = NULL; (l->count)--; @@ -375,11 +375,6 @@ static void *fifo_item_mallocator_new_f(void) return xbt_new(s_xbt_fifo_item_t, 1); } -static void fifo_item_mallocator_free_f(void *item) -{ - xbt_free(item); -} - static void fifo_item_mallocator_reset_f(void *item) { /* memset to zero like calloc */ @@ -398,7 +393,7 @@ XBT_INLINE xbt_fifo_item_t xbt_fifo_new_item(void) */ XBT_INLINE xbt_fifo_item_t xbt_fifo_newitem(void) { - WARN0("This function is deprecated. Use xbt_fifo_new_item."); + XBT_WARN("This function is deprecated. Use xbt_fifo_new_item."); return xbt_fifo_new_item(); } @@ -438,7 +433,7 @@ XBT_INLINE void xbt_fifo_free_item(xbt_fifo_item_t b) */ XBT_INLINE void xbt_fifo_freeitem(xbt_fifo_item_t b) { - WARN0("This function is deprecated. Use xbt_fifo_free_item."); + XBT_WARN("This function is deprecated. Use xbt_fifo_free_item."); xbt_fifo_free_item(b); return; } @@ -455,6 +450,8 @@ XBT_INLINE int xbt_fifo_size(xbt_fifo_t f) /** * \param l a list * \return the head of \a l. + * + * Returns NULL if the list is empty. */ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_first_item(xbt_fifo_t l) { @@ -464,6 +461,8 @@ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_first_item(xbt_fifo_t l) /** * \param l a list * \return the tail of \a l. + * + * Returns NULL if the list is empty. */ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_last_item(xbt_fifo_t l) { @@ -474,13 +473,15 @@ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_last_item(xbt_fifo_t l) */ XBT_INLINE xbt_fifo_item_t xbt_fifo_getFirstItem(xbt_fifo_t l) { - WARN0("This function is deprecated. Use xbt_fifo_get_first_item."); + XBT_WARN("This function is deprecated. Use xbt_fifo_get_first_item."); return xbt_fifo_get_first_item(l); } /** * \param i a bucket * \return the bucket that comes next + * + * Returns NULL if \a i is the tail of the list. */ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_next_item(xbt_fifo_item_t i) { @@ -493,13 +494,15 @@ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_next_item(xbt_fifo_item_t i) */ xbt_fifo_item_t xbt_fifo_getNextItem(xbt_fifo_item_t i) { - WARN0("This function is deprecated. Use xbt_fifo_get_next_item."); + XBT_WARN("This function is deprecated. Use xbt_fifo_get_next_item."); return xbt_fifo_get_next_item(i); } /** * \param i a bucket * \return the bucket that is just before \a i. + * + * Returns NULL if \a i is the head of the list. */ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_prev_item(xbt_fifo_item_t i) { @@ -512,7 +515,7 @@ XBT_INLINE xbt_fifo_item_t xbt_fifo_get_prev_item(xbt_fifo_item_t i) */ xbt_fifo_item_t xbt_fifo_getPrevItem(xbt_fifo_item_t i) { - WARN0("This function is deprecated. Use xbt_fifo_get_prev_item."); + XBT_WARN("This function is deprecated. Use xbt_fifo_get_prev_item."); return xbt_fifo_get_prev_item(i); } @@ -527,7 +530,7 @@ void xbt_fifo_preinit(void) xbt_mallocator_free(item_mallocator); } - item_mallocator = xbt_mallocator_new(256, + item_mallocator = xbt_mallocator_new(65536, fifo_item_mallocator_new_f, fifo_item_mallocator_free_f, fifo_item_mallocator_reset_f);