apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Paul Querna <c...@force-elite.com>
Subject Re: [patch] Add apr_array_clear() API
Date Wed, 16 May 2007 00:18:12 GMT
Daniel Rall wrote:
> The following patch adds a new apr_array_clear() API.  While a trivial
> operation, the minimum (safe) implementation required to clear an
> apr_array_header_t is somewhat non-obvious without examining the
> source code.  Looping over apr_array_pop() is probably a common
> fallback for those who still aren't sure after reading
> apr_array_header_t's doc string.  To clarify things, we should instead
> present an obvious API.
> 
> 
> [[[
> Add apr_array_clear() API.
> 
> * include/apr_tables.h
> * tables/apr_tables.c
>   (apr_array_clear): Declare and define new API.
> 
> * test/testtable.c
>   (a1): Static variable for use across array tests.
>   (array_clear): New test for apr_array_clear().
>   (testtable): Add array_clear() to the test suite.
> ]]]

Committed in r538391. Thanks, sorry for the delay :-)

-Paul

Mime
View raw message