Re: svn commit: r31583 - trunk/subversion/libsvn_wc
On Wed, Jun 4, 2008 at 5:09 PM, Karl Fogel <kfogel_at_red-bean.com> wrote:
> Branko Čibej <brane_at_xbc.nu> writes:
>> And it strikes my that far too much effort is being spent on stylistic
>> nits lately. Yes, consistent style is important, but working code is
>> more important.
> Far too much effort is being spent lately pointing out how much effort
> is being spent on stylistic nits...
> Seriously: I don't see reviews where the patch is rejected solely for
> stylistic nits. Do you? And I try to always make substantive code
> comments along with stylistic ones, or when there are only stylistic
> comments, I go out of my way to point out that the style questions are
> not all that important. I see others doing similarly.
> Gently enforcing stylistic consistency is a perfectly reasonable thing
> for a project to do. If we were to do it in overeager and hard-nosed
> ways -- ways that could waste reviewers' time or drive away potential
> contributors -- then I'd think you were on to something. But, I don't
> see any of that going on. Do you?
FWIW, other than this I do not remember a whole lot of style
conversation going on lately. Also, we have several new committers
and contributors, which is great. So it would not surprise me if some
of the old conversations were being rehashed.
Received on 2008-06-04 23:21:00 CEST
This is an archived mail posted to the Subversion Dev