[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: Peter N. Lundblad <peter_at_famlundblad.se>
Date: 2006-02-04 22:18:39 CET

On Sat, 4 Feb 2006, Bruce DeVisser wrote:

> Peter wrote:
> >Let me first make it clear that this is just for experimentation. This
> >patch is very incomplete, untested and probably buggy.
>
> I assume that means tabs will be supported as well. svn source
> policy is against tabs, but others have to deal with them.
>
Yes, all whitespace characters in the 7-bit ASCII range.

> Sander wrote:
> > >-i --ignore-case (not sure about this; it has encoding problems)
> >
> > What's the benefit of this option? I'm trying to think of a use
> > case...
>
> There are case-insensitive programming languages. Suppose a style
> standardisation (or standards change) is put into effect the same
> as subversion's whitespace standardisation now is: it would be
> nice if blame would work.
>
This is a good point. And to support this is probably as simple as using
apr_toupper and that would work in many cases. My concern is what to do
if/when we support other character encodings on the file content than that
specified by the locale. I'd like us to do that someday; I don't think
it'd be easy to support -i then.

> >
> Sander further wrote:
> > >-b --ignore-space-change (for ASCII whitespace)
> > >-w --ignore-all-space (dito)
> >
> > Can't we just pick one option name?
>
[snip good explanation]

Yes, we need both for different situations as I replied earlier today.

Thansk for your input,
//Peter

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@subversion.tigris.org
For additional commands, e-mail: dev-help@subversion.tigris.org
Received on Sat Feb 4 22:19:10 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.