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

Re: svn commit: r1041230 - /subversion/trunk/subversion/include/svn_checksum.h

From: Blair Zajac <blair_at_orcaware.com>
Date: Mon, 06 Dec 2010 20:16:43 -0800

On 12/6/10 7:47 AM, Julian Foad wrote:
> On Sun, 2010-12-05, Stefan Fuhrmann wrote:
>> I've felt kind of uneasy about that issue as well.
>> However, it would have been difficult to implement
>> a deprecated svn_checksum_to_cstring() in terms
>> of svn_checksum_to_cstring2().
>
> I don't think Blair was concerned about how the function is implemented,
> but about the API promises. (He can correct me if I'm wrong.)

Nope, that's correct.

> But, as I said in my previous reply in this thread, I think it's fine to
> just change the existing API as you have done.

Well, I don't agree, for problems that it could cause later, but don't feel that
strongly to argue about it :)

Blair
Received on 2010-12-07 05:17:26 CET

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