On 13.06.2013 21:41, Oto BREZINA wrote:
> On 2013-06-13 20:57, Stefan Küng wrote:
>> On 13.06.2013 09:29, Oto BREZINA wrote:
>>
>>> Let me elaborate little bit about posibilities so before next rewrite we
>>> will better know what we want to get.
>>>
>>> * Eols and tailing whites - can be detected on the fly
>>> * always asked if detected
>>> * always fixed
>>> * fixed with or without asking if was ok on load - error introduced
>>> by editing in T-Merge, T-Merges purposes in first place is to be
>>> diff/merge tool not an edit, so most of this error comes from other
>>> sources so will be present on load anyway
>>> * fixed with or without asking if was ok on "left" view (not sure
>>> about merge)
>>> Now if you start to build map of all possibly useful version its quite
>>> big thing
>>> My favorite would be:
>>> Auto fix if it was ok in left, ask otherwise.
>> No, auto-fixing without asking is very very bad.
>>
>> Just detect and ask if necessary.
>>
>>> To make it less complex we may add some notification in view title, or
>>> status bar, but it may require lot of processing or very good design.
>>> Lets stick we current approach for a while.
>>>
>>> In spaces and tabs conversion it become even more complex.
>>> Your left file may be all space indented and saving file all tabs. Is
>>> this correct or not ? In current implementation it is correct as it is
>>> self homogeniuos.
>> When doing such fixing, the views have to be handled completely
>> independent of each other. Why would the left view have anything to do
>> with how the right view is saved? Doesn't make any sense to me and IMHO
>> would make things just unnecessarily complicated. And if it's
>> complicated, the users also won't understand why sometimes it asks to
>> fix and sometimes not.
> Few months ago you stand on T-Merge is diff tool where right is editable
> output and left is source.
> I consider left as source thing in MOST cases so can be good source of
> expected tab/spaces style.
> File on load may be damaged by others editing tools (settings), and
> T-Merge as last station before commit can warning you that you are about
> to change that style.
>
> This wont solve fixing left file though.
What does that have to do with keeping the views separate?
I never said that this fixing should only work for one view. I said that
fixing one view has nothing to do with any other view that's visible or
loaded.
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=3058085
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-06-14 17:07:10 CEST