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

RE: Possible TortoiseMerge bug

From: Tobias Kaes <dinonet_at_gmx.de>
Date: Wed, 25 Sep 2013 01:52:33 -0700 (PDT)

> On 24.09.2013 14:29, Tobias Kaes wrote:
> > Hi, I've seen cases where TortoiseMerge doesn't match up left/right
> > sides of changes properly anymore. On the left side it marks lines
> > orange (removed) which are still there on the right side. Seems to be
> > triggered when moving a snipped of code while at the same time
> > replacing it with a smaller snippet. Example files + screenshot
> > attached.
> >
> > Originally observed in 1.8.0 and updating to 1.8.2 didn't help.
>
> Not sure why you think that's a bug.
>
> The lines *are* removed and added again but at another location.
> So TMerge correctly shows that the lines are removed at the original
> location and the new line on the right.
> The lines however are moved further down to a new function, and so
> TMerge shows those lines added again at the new location.
>
> TMerge is a diff tool, it does not know about functions or code or
> whatever. It only knows about lines.

That's alright, I didn't expect it to understand the file content, I'm expecting it to do a line-by-line comparison minimizing the number of edit operations. I've been using TMerge for years and this diff totally caught me off guard (never seen this behaviour before) and in terms of a diff it makes not much sense to me.

I've annotated the screenshot and attached it again to show which lines I mean. Note that the marked orange+white lines did not change at all so I don't see why they can't be white+white in the first place (that's what I would have expected).

I'm reading the colors as remove/insert operations, so orange blocks on the left side have been removed, yellow blocks on the right side have been inserted, gray is padding to keep lines aligned. So orange+yellow means to me "line has been replaced" and orange+gray "line has been deleted" - so what is orange+white supposed to tell me?

------------------------------------------------------
http://tortoisesvn.tigris.org/ds/viewMessage.do?dsForumId=4061&dsMessageId=3065220

To unsubscribe from this discussion, e-mail: [users-unsubscribe_at_tortoisesvn.tigris.org].

screenshot_annotated.png
Received on 2013-09-25 10:52:39 CEST

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

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.