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

Re: svn commit: r32996 - in trunk/subversion: include libsvn_subr

From: Greg Stein <gstein_at_gmail.com>
Date: Tue, 9 Sep 2008 09:46:37 -0700

Of course it doesn't make a difference either way.

Just noting that it would have been easier to review one constify
commit, rather than a dozen.

On Tue, Sep 9, 2008 at 9:22 AM, Karl Fogel <kfogel_at_red-bean.com> wrote:
> "Greg Stein" <gstein_at_gmail.com> writes:
>> Roll back a change? Dude. They are TWO line changes. If somebody
>> doesn't like it, then just apply a change to remove it. "Roll back" is
>> for large changes. These are about as small a change as possible.
>>
>> They are harder to review because you must review a dozen emails. Put
>> those dozen changes into one revision and just go thru it once.
>
> Oy. Does it really make that much of a difference either way? (I mean,
> preference noted -- but we're talking about a pretty minor difference
> here either way...)
>
>

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-09-09 18:46:50 CEST

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