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

Re: TMerge whitespace handling

From: Tobias Schäfer <tobiasschaefer_at_gmx.de>
Date: 2006-03-16 07:35:50 CET

On Wednesday 15 March 2006 23:41, Simon Large wrote:
> Tobias Schäfer wrote:
> >> Does the "ignore ending whitespace" option ignore *only* ending
> >> whitespace, or does it ignore leading whitespace as well?
> >
> > It only ignores *ending* whitespace. This is usefull for languages (eg.
> > python) where the leading spaces influence the control flow but ending
> > whitespace is not relevant, see #233.
>
> Right, but if you want to ignore ending whitespace in python, you
> probably also want to ignore it in other languages too. At the moment
> you can't ignore leading AND ending whitespace.

Now I get it: it is possible to ignore leading and ending whitespace by
choosing "Ignore all whitespace, but then not *only* leading and ending but
also inline whitespace gets ignored.

> >> Should these perhaps be checkboxes so you can choose to ignore
> >> leading/ending/inline spaces independently?
> >
> > Three checkboxes could actually replace the four radio buttons and offer
> > even more combinations:
> > [ ] Ignore leading whitespace
> > [ ] Ignore inline whitespace
> > [ ] Ignore ending whitespace
> >
> > I'm not sure if this is needed though.
>
> No, probably not. I'm just being picky and looking for inconsistencies ;-)

Anyone out there who needs it?

It cannot be that hard to implement. But we'll need to think about if and how
we map existing configurations to the new checkboxes.
The easiest solution: define a new registry key with a new default value.
Users which have changed the default radio button in a previous installation
might be bothered though.

Tobias

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Thu Mar 16 07:46:03 2006

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.