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

RE: [TSVN] Diff / Merge settings

From: Lübbe Onken <l.onken_at_rac.de>
Date: 2005-02-09 15:03:48 CET

Hi Steve,

> I think you should know (or remember). The reason for this is that
> some people want to use other external diff/merge tools, but still be
> able to switch fast between the external and TMerge.
I know, I do this regularly and I think the current implementation is messy.
That's what the checkboxes are for. If the checkboxes just place a comment
(well sometimes :-) ) in front of the commandline(s) and are just used to
enable / disable the commandline(s), what are they good for? We could get
rid of them entirely and just use the comment character, but then we have to
answer questions on how to enable diff viewers over and over again.

> So we need a way
> to 'deactivate' the external diff/merge program _without_ deleting it
> completely (otherwise you'd have to enter the path and the params all
> over again).
And why don't you just spend three registry entries: ExternalDiff,
ExternalMerge and ExternalViewer which are set to 0 or 1 depending on the
checkbox? You have a clean assignment of UI element to variable and no
messing around with strings.

Cheers
- Lübbe

--
       ___
  oo  // \\      "De Chelonian Mobile"
 (_,\/ \_/ \     TortoiseSVN
   \ \_/_\_/>    The coolest Interface to (Sub)Version Control
   /_/   \_\     http://tortoisesvn.tigris.org
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tortoisesvn.tigris.org
For additional commands, e-mail: dev-help@tortoisesvn.tigris.org
Received on Wed Feb 9 15:04:34 2005

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.