Re: Line diff handled badly (by old algorithm)

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