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

Re: svn commit: rev 5237 - in branches/issue-650-ssl-certs/subversion: clients/cmdline libsvn_ra_dav libsvn_subr

From: Philip Martin <philip_at_codematters.co.uk>
Date: 2003-03-07 12:59:39 CET

mass@tigris.org writes:

> Author: mass
> Date: Fri Mar 7 02:06:55 2003
> New Revision: 5237
>
> Modified:
> branches/issue-650-ssl-certs/subversion/clients/cmdline/main.c
> branches/issue-650-ssl-certs/subversion/libsvn_ra_dav/session.c
> branches/issue-650-ssl-certs/subversion/libsvn_subr/config_file.c
> Log:
> libsvn_subr/config_file, libsvn_ra_dav/session.c, clients/cmdline/main.c : Convert tabs to spaces

It's generally a bad idea to make unnecessary white-space changes on a
branch. It increases the size of the trunk-branch diff making it
harder to review the branch as a whole. It also increases the chances
of getting a conflict when you come to merge the branch back to the
trunk. It's perfectly OK to change white-space as part of a change to
a line that is intrinsic to the branch, or as part of a merge from the
trunk to the branch.

-- 
Philip Martin
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Fri Mar 7 13:00:27 2003

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.