X-Git-Url: https://git.sesse.net/?a=blobdiff_plain;f=include%2Fvlc_arrays.h;h=b8dbae473ccc7f428c7f79a3ed9ff90fb2eae9aa;hb=3118165aefce584687fd67953e477fbbdf7e1bcb;hp=a121000f2d3705ade9f0b18fd62f8f6a7472f83e;hpb=05c8c0d9164b349c8fe94386b4e5270c5d039339;p=vlc diff --git a/include/vlc_arrays.h b/include/vlc_arrays.h index a121000f2d..b8dbae473c 100644 --- a/include/vlc_arrays.h +++ b/include/vlc_arrays.h @@ -30,6 +30,13 @@ * This file defines functions, structures and macros for handling arrays in vlc */ +/* realloc() that never fails *if* downsizing */ +static inline void *realloc_down( void *ptr, size_t size ) +{ + void *ret = realloc( ptr, size ); + return ret ? ret : ptr; +} + /** * Simple dynamic array handling. Array is realloced at each insert/removal */ @@ -41,37 +48,36 @@ #define INSERT_ELEM( p_ar, i_oldsize, i_pos, elem ) \ do \ { \ - if( !i_oldsize ) (p_ar) = NULL; \ + if( !(i_oldsize) ) (p_ar) = NULL; \ (p_ar) = VLCCVP realloc( p_ar, ((i_oldsize) + 1) * sizeof(*(p_ar)) ); \ + if( !(p_ar) ) abort(); \ if( (i_oldsize) - (i_pos) ) \ { \ memmove( (p_ar) + (i_pos) + 1, (p_ar) + (i_pos), \ ((i_oldsize) - (i_pos)) * sizeof( *(p_ar) ) ); \ } \ - (p_ar)[i_pos] = elem; \ + (p_ar)[(i_pos)] = elem; \ (i_oldsize)++; \ } \ while( 0 ) -#define REMOVE_ELEM( p_ar, i_oldsize, i_pos ) \ +#define REMOVE_ELEM( p_ar, i_size, i_pos ) \ do \ { \ - if( (i_oldsize) - (i_pos) - 1 ) \ + if( (i_size) - (i_pos) - 1 ) \ { \ memmove( (p_ar) + (i_pos), \ (p_ar) + (i_pos) + 1, \ - ((i_oldsize) - (i_pos) - 1) * sizeof( *(p_ar) ) ); \ - } \ - if( i_oldsize > 1 ) \ - { \ - (p_ar) = realloc( p_ar, ((i_oldsize) - 1) * sizeof( *(p_ar) ) ); \ + ((i_size) - (i_pos) - 1) * sizeof( *(p_ar) ) ); \ } \ + if( i_size > 1 ) \ + (p_ar) = realloc_down( p_ar, ((i_size) - 1) * sizeof( *(p_ar) ) );\ else \ { \ free( p_ar ); \ (p_ar) = NULL; \ } \ - (i_oldsize)--; \ + (i_size)--; \ } \ while( 0 ) @@ -94,6 +100,7 @@ (tab) = cast realloc( tab, sizeof( void ** ) * ( (count) + 1 ) ); \ else \ (tab) = cast malloc( sizeof( void ** ) ); \ + if( !(tab) ) abort(); \ (tab)[count] = (p); \ (count)++; \ } while(0) @@ -144,6 +151,7 @@ (tab) = cast realloc( tab, sizeof( void ** ) * ( (count) + 1 ) ); \ else \ (tab) = cast malloc( sizeof( void ** ) ); \ + if( !(tab) ) abort(); \ if( (count) - (index) > 0 ) \ memmove( (void**)(tab) + (index) + 1, \ (void**)(tab) + (index), \ @@ -192,6 +200,7 @@ (array).i_alloc = newsize; \ (array).p_elems = VLCCVP realloc( (array).p_elems, (array).i_alloc * \ sizeof(*(array).p_elems) ); \ + if( !(array).p_elems ) abort(); \ } #define _ARRAY_GROW1(array) { \ @@ -405,51 +414,56 @@ static inline uint64_t DictHash( const char *psz_string, int hashsize ) return i_hash % hashsize; } -struct vlc_dictionary_entry_t +typedef struct vlc_dictionary_entry_t { char * psz_key; void * p_value; struct vlc_dictionary_entry_t * p_next; -}; +} vlc_dictionary_entry_t; typedef struct vlc_dictionary_t { int i_size; - struct vlc_dictionary_entry_t ** p_entries; + vlc_dictionary_entry_t ** p_entries; } vlc_dictionary_t; static void * const kVLCDictionaryNotFound = NULL; static inline void vlc_dictionary_init( vlc_dictionary_t * p_dict, int i_size ) { + p_dict->p_entries = NULL; + if( i_size > 0 ) { - p_dict->p_entries = (struct vlc_dictionary_entry_t **)malloc(sizeof(struct vlc_dictionary_entry_t *) * i_size); - memset( p_dict->p_entries, 0, sizeof(struct vlc_dictionary_entry_t *) * i_size ); + p_dict->p_entries = (vlc_dictionary_entry_t **)calloc( i_size, sizeof(*p_dict->p_entries) ); + if( !p_dict->p_entries ) + i_size = 0; } - else - p_dict->p_entries = NULL; p_dict->i_size = i_size; } -static inline void vlc_dictionary_clear( vlc_dictionary_t * p_dict ) +static inline void vlc_dictionary_clear( vlc_dictionary_t * p_dict, + void ( * pf_free )( void * p_data, void * p_obj ), + void * p_obj ) { - int i; - struct vlc_dictionary_entry_t * p_current, * p_next; if( p_dict->p_entries ) { - for( i = 0; i < p_dict->i_size; i++ ) + for( int i = 0; i < p_dict->i_size; i++ ) { + vlc_dictionary_entry_t * p_current, * p_next; p_current = p_dict->p_entries[i]; while( p_current ) { p_next = p_current->p_next; + if( pf_free != NULL ) + ( * pf_free )( p_current->p_value, p_obj ); free( p_current->psz_key ); free( p_current ); p_current = p_next; } } free( p_dict->p_entries ); + p_dict->p_entries = NULL; } p_dict->i_size = 0; } @@ -463,7 +477,7 @@ vlc_dictionary_value_for_key( const vlc_dictionary_t * p_dict, const char * psz_ return kVLCDictionaryNotFound; int i_pos = DictHash( psz_key, p_dict->i_size ); - struct vlc_dictionary_entry_t * p_entry = p_dict->p_entries[i_pos]; + vlc_dictionary_entry_t * p_entry = p_dict->p_entries[i_pos]; if( !p_entry ) return kVLCDictionaryNotFound; @@ -481,7 +495,7 @@ vlc_dictionary_value_for_key( const vlc_dictionary_t * p_dict, const char * psz_ static inline int vlc_dictionary_keys_count( const vlc_dictionary_t * p_dict ) { - struct vlc_dictionary_entry_t * p_entry; + vlc_dictionary_entry_t * p_entry; int i, count = 0; if( !p_dict->p_entries ) @@ -497,7 +511,7 @@ vlc_dictionary_keys_count( const vlc_dictionary_t * p_dict ) static inline char ** vlc_dictionary_all_keys( const vlc_dictionary_t * p_dict ) { - struct vlc_dictionary_entry_t * p_entry; + vlc_dictionary_entry_t * p_entry; char ** ppsz_ret; int i, count = vlc_dictionary_keys_count( p_dict ); @@ -521,9 +535,9 @@ __vlc_dictionary_insert( vlc_dictionary_t * p_dict, const char * psz_key, vlc_dictionary_init( p_dict, 1 ); int i_pos = DictHash( psz_key, p_dict->i_size ); - struct vlc_dictionary_entry_t * p_entry; + vlc_dictionary_entry_t * p_entry; - p_entry = (struct vlc_dictionary_entry_t *)malloc(sizeof(struct vlc_dictionary_entry_t)); + p_entry = (vlc_dictionary_entry_t *)malloc(sizeof(*p_entry)); p_entry->psz_key = strdup( psz_key ); p_entry->p_value = p_value; p_entry->p_next = p_dict->p_entries[i_pos]; @@ -532,7 +546,8 @@ __vlc_dictionary_insert( vlc_dictionary_t * p_dict, const char * psz_key, { /* Count how many items there was */ int count; - for( count = 1; p_entry->p_next; count++ ) p_entry = p_entry->p_next; + for( count = 1; p_entry->p_next; count++ ) + p_entry = p_entry->p_next; if( count > 3 ) /* XXX: this need tuning */ { /* Here it starts to be not good, rebuild a bigger dictionary */ @@ -547,12 +562,12 @@ __vlc_dictionary_insert( vlc_dictionary_t * p_dict, const char * psz_key, { __vlc_dictionary_insert( &new_dict, p_entry->psz_key, p_entry->p_value, - 0 /* To avoid multiple rebuild loop */); + false /* To avoid multiple rebuild loop */); p_entry = p_entry->p_next; } } - vlc_dictionary_clear( p_dict ); + vlc_dictionary_clear( p_dict, NULL, NULL ); p_dict->i_size = new_dict.i_size; p_dict->p_entries = new_dict.p_entries; } @@ -562,18 +577,20 @@ __vlc_dictionary_insert( vlc_dictionary_t * p_dict, const char * psz_key, static inline void vlc_dictionary_insert( vlc_dictionary_t * p_dict, const char * psz_key, void * p_value ) { - __vlc_dictionary_insert( p_dict, psz_key, p_value, 1 ); + __vlc_dictionary_insert( p_dict, psz_key, p_value, true ); } static inline void -vlc_dictionary_remove_value_for_key( const vlc_dictionary_t * p_dict, const char * psz_key ) +vlc_dictionary_remove_value_for_key( const vlc_dictionary_t * p_dict, const char * psz_key, + void ( * pf_free )( void * p_data, void * p_obj ), + void * p_obj ) { if( !p_dict->p_entries ) return; int i_pos = DictHash( psz_key, p_dict->i_size ); - struct vlc_dictionary_entry_t * p_entry = p_dict->p_entries[i_pos]; - struct vlc_dictionary_entry_t * p_prev; + vlc_dictionary_entry_t * p_entry = p_dict->p_entries[i_pos]; + vlc_dictionary_entry_t * p_prev; if( !p_entry ) return; /* Not found, nothing to do */ @@ -583,6 +600,8 @@ vlc_dictionary_remove_value_for_key( const vlc_dictionary_t * p_dict, const char do { if( !strcmp( psz_key, p_entry->psz_key ) ) { + if( pf_free != NULL ) + ( * pf_free )( p_entry->p_value, p_obj ); if( !p_prev ) p_dict->p_entries[i_pos] = p_entry->p_next; else