Changeset 4817

Show
Ignore:
Timestamp:
06/04/07 18:24:31 (1 year ago)
Author:
morris
Message:

Clean up a bit, add some new assertions

Files:

Legend:

Unmodified
Added
Removed
Modified
Copied
Moved

  • libwired/trunk/libwired/collections/wi-array.c

    r4800 r4817  
    7676#define _WI_ARRAY_INDEX_ASSERT(array, index)                                    \ 
    7777        WI_ASSERT((index) < (array)->data_count,                                        \ 
    78                 "index %d out of range (count %d) in %@",                             \ 
     78                "index %ld out of range (count %lu) in %@",                           \ 
    7979                (index), (array)->data_count, (array)) 
    8080 
    …… 
    8686}; 
    8787typedef struct _wi_array_item                   _wi_array_item_t; 
     88 
    8889 
    8990struct _wi_array { 
    …… 
    116117static void                                                             _wi_array_optimize(wi_array_t *); 
    117118 
    118 static _wi_array_item_t *                               _wi_array_item_create(wi_array_t *); 
    119 static void                                                             _wi_array_item_remove(wi_array_t *, _wi_array_item_t *); 
     119static _wi_array_item_t *                               _wi_array_create_item(wi_array_t *); 
     120static void                                                             _wi_array_remove_item(wi_array_t *, _wi_array_item_t *); 
    120121static void                                                             _wi_array_add_item(wi_array_t *, _wi_array_item_t *); 
    121122static void                                                             _wi_array_add_item_sorted(wi_array_t *, _wi_array_item_t *, wi_compare_func_t *); 
    …… 
    519520 
    520521wi_boolean_t wi_array_contains_data(wi_array_t *array, void *data) { 
    521         wi_uinteger_t   i; 
    522  
    523         for(i = 0; i < array->data_count; i++) { 
    524                 if(_WI_ARRAY_IS_EQUAL(array, array->items[i]->data, data)) 
    525                         return true; 
    526         } 
    527  
    528         return false; 
     522        return (wi_array_index_of_data(array, data) != WI_NOT_FOUND); 
    529523} 
    530524 
    …… 
    706700#pragma mark - 
    707701 
    708 static _wi_array_item_t * _wi_array_item_create(wi_array_t *array) { 
     702static _wi_array_item_t * _wi_array_create_item(wi_array_t *array) { 
    709703        _wi_array_item_t        *item, *item_block; 
    710704        size_t                          size; 
    …… 
    736730 
    737731 
    738 static void _wi_array_item_remove(wi_array_t *array, _wi_array_item_t *item) { 
     732static void _wi_array_remove_item(wi_array_t *array, _wi_array_item_t *item) { 
    739733        _WI_ARRAY_RELEASE(array, item->data); 
    740734        item->data = NULL; 
    …… 
    780774                _wi_array_grow(array, index); 
    781775         
    782         memcpy(array->items + index + 1, 
    783                    array->items + index, 
    784                    (array->data_count - index) * sizeof(_wi_array_item_t *)); 
     776        memmove(array->items + index + 1, 
     777                       array->items + index, 
     778                       (array->data_count - index) * sizeof(_wi_array_item_t *)); 
    785779         
    786780        array->items[index] = item; 
    …… 
    793787 
    794788static int _wi_array_compare_data(void *context, const void *p1, const void *p2) { 
    795         return (*(wi_compare_func_t *) context)(*(void **) p1, *(void **)p2); 
     789        return (*(wi_compare_func_t *) context)(*(void **) p1, *(void **) p2); 
    796790} 
    797791 
    …… 
    799793 
    800794static int _wi_array_compare_data(const void *p1, const void *p2) { 
    801         return (*_wi_array_sort_function)(*(void **) p1, *(void **)p2); 
     795        return (*_wi_array_sort_function)(*(void **) p1, *(void **) p2); 
    802796} 
    803797 
    …… 
    856850        _wi_array_item_t        *item; 
    857851 
    858         item = _wi_array_item_create(array); 
     852        item = _wi_array_create_item(array); 
    859853        item->data = _WI_ARRAY_RETAIN(array, data); 
    860854 
    …… 
    867861        _wi_array_item_t        *item; 
    868862 
    869         item = _wi_array_item_create(array); 
     863        item = _wi_array_create_item(array); 
    870864        item->data = _WI_ARRAY_RETAIN(array, data); 
    871865 
    …… 
    891885        _WI_ARRAY_INDEX_ASSERT(array, index); 
    892886 
    893         item = _wi_array_item_create(array); 
     887        item = _wi_array_create_item(array); 
    894888        item->data = _WI_ARRAY_RETAIN(array, data); 
    895889 
    …… 
    904898        _WI_ARRAY_INDEX_ASSERT(array, index); 
    905899 
    906         item = _wi_array_item_create(array); 
     900        item = _wi_array_create_item(array); 
    907901        item->data = _WI_ARRAY_RETAIN(array, data); 
    908902         
    909         _wi_array_item_remove(array, array->items[index]); 
     903        _wi_array_remove_item(array, array->items[index]); 
    910904        array->items[index] = item; 
    911905} 
    …… 
    944938        _WI_ARRAY_INDEX_ASSERT(array, index); 
    945939 
    946         _wi_array_item_remove(array, array->items[index]); 
     940        _wi_array_remove_item(array, array->items[index]); 
    947941 
    948942        memcpy(array->items + index, 
    …… 
    973967 
    974968void wi_array_remove_all_data(wi_array_t *array) { 
    975         wi_uinteger_t   i, count; 
     969        wi_uinteger_t           i, count; 
     970        _wi_array_item_t        *item; 
    976971         
    977972        count = array->data_count; 
    978973         
    979974        if(count > 0) { 
    980                 for(i = 0; i < count; i++) 
    981                         _wi_array_item_remove(array, array->items[i]); 
     975                for(i = 0; i < count; i++) { 
     976                        item = array->items[i]; 
     977                         
     978                        WI_ASSERT(item != NULL, "NULL item at index %lu (count %lu)", 
     979                                i, count); 
     980 
     981                        _wi_array_remove_item(array, item); 
     982                } 
    982983                 
    983984                array->data_count = 0; 

  • libwired/trunk/test/tests/wi-crypto-tests.c

    r4697 r4817  
    6464         
    6565        cipher = wi_autorelease(wi_cipher_init_with_key(wi_cipher_alloc(), type, key, iv)); 
    66                                                         
     66         
    6767        WI_TEST_ASSERT_NOT_NULL(cipher, ""); 
    6868        WI_TEST_ASSERT_EQUAL_INSTANCES(wi_cipher_key(cipher), key, ""); 

  • libwired/trunk/test/tests/wi-p7-spec-tests.c

    r4701 r4817  
    3333 
    3434 
     35#ifdef WI_P7 
    3536static char                                                             *_wi_test_p7_spec1 = 
    3637        "<?xml version=\"1.0\" encoding=\"UTF-8\"?>" 
    …… 
    6364        "       </p7:messages>" 
    6465        "</p7:protocol>"; 
     66#endif 
    6567 
    6668