[asterisk-dev] [asterisk-commits] murf: trunk r89591 - /trunk/main/hashtab.c

[Kevin P. Fleming]

SVN commits to the Asterisk project wrote:

> -struct ast_hashtab *ast_hashtab_create(int initial_buckets,
> -	int (*compare)(const void *a, const void *b), /* a func to compare two elements in the hash -- cannot be null  */
> -	int (*resize)(struct ast_hashtab *), /* a func to decide if the table needs to be resized, a NULL ptr here will cause a default to be used */
> -	int (*newsize)(struct ast_hashtab *tab), /* a ptr to func that returns a new size of the array. A NULL will cause a default to be used */
> -	unsigned int (*hash)(const void *obj), /* a func to do the hashing */
> -	int do_locking ) /* use locks to guarantee safety of iterators/insertion/deletion -- real simpleminded right now */
> +struct ast_hashtab *ast_hashtab_create(int initial_buckets, 
> +	int (*compare)(const void *a, const void *b), 
> +	int (*resize)(struct ast_hashtab *), 
> +	int (*newsize)(struct ast_hashtab *tab),
> +	unsigned int (*hash)(const void *obj), 
> +	int do_locking)
>  {
>  	struct ast_hashtab *ht;
>  	

This hunk removes the documentation of the arguments... was this
intended? They should be put back, and using Doxygen syntax as the
coding guidelines mandate.

-- 
Kevin P. Fleming
Director of Software Technologies
Digium, Inc. - "The Genuine Asterisk Experience" (TM)