After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 362233 - Manual sync points for diff
Manual sync points for diff
Status: RESOLVED FIXED
Product: meld
Classification: Other
Component: filediff
unspecified
Other Linux
: Normal normal
: ---
Assigned To: Stephen Kennedy
Stephen Kennedy
: 588092 (view as bug list)
Depends on:
Blocks:
 
 
Reported: 2006-10-14 22:45 UTC by Sven Herzberg
Modified: 2013-05-19 13:47 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
Diff backend patch (5.25 KB, patch)
2013-03-24 02:35 UTC, Kai Willadsen
none Details | Review
PEP fixes (2.52 KB, patch)
2013-03-24 02:35 UTC, Kai Willadsen
none Details | Review
Horrendously dodgy UI for split feature (6.03 KB, patch)
2013-03-24 02:36 UTC, Kai Willadsen
none Details | Review

Description Sven Herzberg 2006-10-14 22:45:13 UTC
I'd like to be able to give some advice to the diff routines. Somthing like "a piece of text starts and ands at these two positions" for each file.

This would produce a lot better results for several use cases.
Comment 1 Kai Willadsen 2009-07-08 22:00:13 UTC
*** Bug 588092 has been marked as a duplicate of this bug. ***
Comment 2 Kai Willadsen 2009-07-08 22:11:05 UTC
Just a quick note on what this would (at a guess) entail...

FileDiff would need to be able to have multiple Differs mapped to different sections of text, editing text would have to dispatch changes to the correct Differ, and then the results of all Differs would need to be merged before being passed on to the drawing methods. And, of course, there would also need to be a reasonable UI for setting sync points.
Comment 3 Stephen Kennedy 2009-07-11 12:15:40 UTC
Araxis does this quite well - you right click in the area between the text areas and it inserts an arrow between the two sides. The arrow heads can then be dragged to the appropriate line. I don't think there's a UI for keyboardists except to type the two line numbers.

As Kai says, the "sync point" is more of a "split point". Effectively you get two different differs for above and below the sync point.
Comment 4 Kai Willadsen 2013-03-24 02:35:31 UTC
Created attachment 239655 [details] [review]
Diff backend patch
Comment 5 Kai Willadsen 2013-03-24 02:35:56 UTC
Created attachment 239656 [details] [review]
PEP fixes
Comment 6 Kai Willadsen 2013-03-24 02:36:23 UTC
Created attachment 239657 [details] [review]
Horrendously dodgy UI for split feature
Comment 7 Kai Willadsen 2013-04-03 21:06:33 UTC
These patches and some additional bits have been pushed to git (hash 659649 and onwards) and will show up in 1.7.2.

Thanks for the bug report.