Re: [PATCH] remove svn__strtol() and svn__strtoul()

On 07.08.2014 08:52, Stefan Sperling wrote:
> On Thu, Aug 07, 2014 at 01:34:40AM +0200, Branko Čibej wrote:
>> On 06.08.2014 23:23, Stefan Sperling wrote:
>>> How about I commit my patch, and then someone else (you? Bert?) tunes
>>> the implementations of svn_cstring_atoi*, perhaps reusing code from
>>> svn__strto* but keeping the API promises intact?
>> You'd be silently changing the semantics of public APIs. These are not
>> currently documented to be equivalent to strtol & co., but they're
>> implemented in terms of apr_strtol & co., which are just replacements
>> for those platforms that do not have the standard <string.h> header.
> So?

So we can't go silently changing the semantics of our public APIs.

Unless you can convince someone (me?) that this is a topic for API errata.

-- Brane

-- 
Branko Čibej | Director of Subversion
WANdisco | Realising the impossibilities of Big Data
e. brane_at_wandisco.com