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

Re: svn commit: r995475 - /subversion/trunk/subversion/libsvn_repos/load.c

From: Hyrum K. Wright <hyrum_wright_at_mail.utexas.edu>
Date: Wed, 15 Sep 2010 21:06:26 +0200

On Wed, Sep 15, 2010 at 9:03 PM, Stefan Sperling <stsp_at_elego.de> wrote:
> On Fri, Sep 10, 2010 at 04:02:52PM +0200, Branko Čibej wrote:
>>  On 10.09.2010 15:26, Stefan Sperling wrote:
>> > I know we're using C89, but maybe it's time to move on and upgrade to C99
>> > where the benefits are desirable? When Subversion was started, C89 was about
>> > a decade old, and C99 is just as old now...
>>
>> Microsoft's C compiler, to name only one, still does not provide most of
>> the C99 features. I don't know about standard library support.
>>
>> However, I don't see where you gain with using strtol(). First of all,
>> it was already in C89 and has effectively the same interface as the APR
>> conversions. C99 added strtoll() but it has the same interface. What's
>> the benefit?
>
> Raising this thread again, because there is a benefit to using strtol()'s
> unsigned cousin, strtoul(). APR does not provide an interface for it.
> Can we use it? Can we also use its 64-bit cousin strtoull()? If we can use
> those two, we might as well use strtol() and strtoul() and skip apr_strtoi64()
> completely.

Adding an implementation to APR isn't a problem, but it would take a
while to trickle down (we'd still need a private implementation).

-Hyrum
Received on 2010-09-15 21:07:05 CEST

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.