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

Re: [PATCH] Experimental svn diff -w

From: Sander Striker <striker_at_apache.org>
Date: 2006-02-04 13:29:10 CET

Jim Blandy wrote:
> On 2/3/06, Peter N. Lundblad <peter@famlundblad.se> wrote:
>
>>I've implemented a line normalization function, which currently just drops
>>all space characters.
>
> Have you tried a normalization function which simply replaces all
> whitespace sequences with a single space? Deleting whitespace in most
> programming languages concatenates identifiers, which is a change
> people would usually like to know about.

Yes, agreed. Basically you can get away with removing all whitespace at
the start and end of the line, and compressing all other whitespace to
a single space.

> Of course, that wouldn't help us in our current space-before-paren
> situation.

No, that's true. And this is something to keep in mind. Do we wish
to ignore all whitespace? Or do we want to ignore all changes within
whitespace; this excludes addition of whitespace where there was no
whitespace before and the removal of all whitespace.
Maybe both options should be available.

Sander

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Feb 4 14:35:56 2006

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.