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

Re: svn commit: r15423 - in trunk/subversion: libsvn_ra_dav

From: Peter McNab <mcnab_p_at_melbpc.org.au>
Date: 2005-07-26 04:43:27 CEST

kfogel@collab.net wrote:

snip (...)

>It looks like part of this (in the 501 trap code) is just a formatting
>fix -- you removed a space before a paren. I guess ideally that
>formatting fix would be its own commit, but if it's going to be part
>of this commit, at least mention it in the log message, so the reader
>knows to expect it.
>
>Nice change otherwise!
>
>-Karl
>
>
>
There has been quite a discussion about adhering to an agreed source
formatting standard and the hassles that arise from conformance.
Maybe it's time to adopt a code formatting tool that everyone can use,
or one that get's used when patches are generated.
Take the mechanics out of the process so people can concentrate on the
real work.

My 2c worth

Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Tue Jul 26 04:44:24 2005

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.