apr-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Greg Stein <gst...@lyra.org>
Subject Re: cvs commit: apr/tables apr_hash.c
Date Sat, 20 Jan 2001 10:28:05 GMT
On Tue, Jan 16, 2001 at 01:56:18PM -0500, Jeff Trawick wrote:
>...
> > > >   +APR_DECLARE(apr_size_t) apr_hash_count(apr_hash_t *ht);
> > > 
> > > silly question: why not int or apr_int32_t instead of apr_size_t?
> > 
> > apr_size_t is the internal type. change that, and you can change the return
> > type :-)
> 
> okay, just checking...  maybe I'll do just that :)
> 
> (IMHO, a sane API is more important than matching internal type,
> though it seems that both are easy to accomplish in this case)

I'd suggest leaving a note for yourself in STATUS or you're gonna forget
this one :-)

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/

Mime
View raw message