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

Re: Line diff handled badly (by old algorithm)

From: Stefan Küng <tortoisesvn_at_gmail.com>
Date: 2006-06-06 08:39:52 CEST

On 6/6/06, Simon Large <simon@skirridsystems.co.uk> wrote:

> I like it :-) It has certainly fixed the spurious matching in the old
> algorithm.
>
> I'm not sure the line diff algorithm is correct yet though, or at least
> the colour coding. Try these 2 lines:
>
> void EventWrite(U8 EventType, U8 Device)
> void EventWrite(U8 EventType, U8 Ident)
>
> I get some very odd colour coding rather than a straight delete of
> 'Device' and an add of 'Ident'.

Well, it is 'correct'. The problem is that in such cases, it's hard
for the algorithm to know if it would be better to just show a
complete replace or still try to show inline (in this case: inword)
diffs.

Stefan

-- 
       ___
  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 Tue Jun 6 08:40:05 2006

This is an archived mail posted to the TortoiseSVN Dev mailing list.