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

Re: svn commit: r1586947 - in /subversion/trunk/subversion/libsvn_fs_fs: dag.c dag.h tree.c

From: Julian Foad <julianfoad_at_btopenworld.com>
Date: Mon, 14 Apr 2014 18:39:42 +0100 (BST)

Ivan Zhakov wrote:
> Julian Foad wrote:
>> It seems the main problem here is simply that this log message summary line gives
>> a false impression about the magnitude of this particular change.
>> I totally support this particular kind of change. It's simply good interface design.
> I completely agree with Julian: the change itself is good, but
> performance should not be justification for it.

I don't want to drag out this thread any longer, but to make sure my position is clear: I think
performance IS a sufficient justification. I also think design style would be a sufficient justification. Either of those alone or both together would be sufficient. (And they are not entirely separate.)

- Julian

> It makes design more
> clear and consistent with other function that have IGNORE_ENOENT
> argument.
Received on 2014-04-14 19:40:19 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.