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

Re: T-Merge Save as behaviour

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: Sat, 06 Apr 2013 23:21:00 +0200

On 06.04.2013 21:56, Oto BREZINA wrote:
> On 2013-04-06 21:42, Stefan Küng wrote:
>> On 06.04.2013 21:10, Oto BREZINA wrote:
>>> I'm continuing in "Multi-view edit".
>>>
>>> I plan to add question what file to "save as" even only one file is
>>> editable.
>> you mean you want to ask where to save the file all the time?
>> Please don't do that.
> I meant ask What file (view) to "save as".
>>
>>> I found out that using "save as" file can get resolved. Is this really
>>> good idea? Whag file is resoulved (one with old name or with new name?)
>> What do you mean by "resolved"?
> here is code "snip":
>
> bool CMainFrame::FileSaveAs(bool bCheckResolved /*=true*/)
> {
> if (bCheckResolved && HasConflictsWontKeep())
> return false;
>
> CString fileName;
> if(!TryGetFileName(fileName))
> return false;
>
> SaveFile(fileName);
> return true;
> }
>
>
> first if is making sure user know there are unresolved changes ...
>
> SaveFile call allows to mark resolved
>
> Note: I did not try that - just reading code

That check ask the user whether (s)he's sure if (s)he want's to save the
file with known conflicts.
Because if there are conflicts, the file is save *with* the conflict
markers still in place - and those markers are not shown in TMerge
(TMerge shows the conflicts but differently).
So I think that check should stay.

>>> From my point of view when you "save as" original conflicted file is
>>> not resolved. Did I miss something?
>> No, when you save the file to a different name/location, the original
>> file is not marked as resolved because it isn't resolved - the saved
>> file will be but not the file that has the conflicts.
> So :
> 1. is mark as resolved shown in this case ?

No.

> 2. is target file (with new name) marked as resolved - what ever state
> it had before. (e.g. TortoiseProc is called for it?)

No, because it isn't resolved.

> 3. you start to add GIT I guess there are also conflicts, will T-Merge
> support that?

Not sure if I add Git support to TSVN - I'm working on a different
project now...

>>> Instead of ask user to resolve any remaining blocks, I plan to simply
>>> note it in displayed options dialog.
>> Nope, don't do that. It's important that files get resolved.
> Only for "save as" dialog; I'll try my best ... adding default option
> continue resolving would do that?

When 'save as' is done, user isn't asked whether to mark the conflict as
resolved.

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=3052697
To unsubscribe from this discussion, e-mail: [dev-unsubscribe_at_tortoisesvn.tigris.org].
Received on 2013-04-06 23:21:13 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.