[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

Re: CVS update: subversion/subversion/libsvn_subr hashdump.c

From: Greg Stein <gstein_at_lyra.org>
Date: 2000-11-07 01:41:31 CET

On Tue, Nov 07, 2000 at 12:55:45AM +0100, Branko Cibej wrote:
> Karl Fogel wrote:
> >> By the way, shouldn't that parameter be `apr_size_t'?
> >
> > I'm not really sure what the advantage/disadvantage would be. (?)
>
> Oh well, no advantage as such, just consistency. Do we want to be
> consistent about using APR types? I'd say yes, if it doesn't cost a lot,
> but It's not my call.

apr_size_t exists in case some whacko platform doesn't have a size_t in its
header files. Dunno any, but the need was discovered by Apache (at some
point in the past) and it just flowed into APR.

My guess is that it isn't really needed any more.

I'm with Branko on this one: let's use apr_[s]size_t consistently.

Cheers,
-g

-- 
Greg Stein, http://www.lyra.org/
Received on Sat Oct 21 14:36:14 2006

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.