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

Re: [tortoisesvn] r24360 committed - WhiteSpace-Fix handling:...

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Fri, 14 Jun 2013 18:48:41 +0200

On 14.06.2013 18:01, Oto BREZINA wrote:

> Just: not long ago T-Merge philosofy was Left is source, right is
> target/edit (in diff). Using this : formating left is more possible
> proper format according repository style against right one.
> e.g. you are diffing .rc file where spaces are on line ends, T-Merge can
> see it is that way on repository, so will offer to fix it. But if you
> indent by tabs it can detect it ...
> That's all I meant. Maybe it was incorrect deduction but it worked in my
> mind.

But who's to say that the left (repository, working base, ...) is
correct? I'd say in most situations it is not correct either if the
right view is not.
So really the views should be treated separately and only the one to
save (if we keep the on-save detection) or the active/editable view
should be considered, no other view should be mixed in.

Stefan

-- 
        ___
   oo  // \\      "De Chelonian Mobile"
  (_,\/ \_/ \     TortoiseSVN
    \ \_/_\_/>    The coolest interface to (Sub)version control
    /_/   \_\     http://tortoisesvn.net
------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=757&dsMessageId=3058096
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-06-14 18:48:51 CEST

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.