many 'diff' implementations

Bug #88861 reported by Robert Collins
2
Affects Status Importance Assigned to Milestone
Bazaar
Confirmed
Medium
Unassigned
Breezy
Triaged
Low
Unassigned

Bug Description

we have too many diff routines.

A current incomplete list is:
InterTree.compare
builtins.cmd_renames
buildins.cmd_deleted
builtins.cmd_added
commit
merge.Merge3Merger.__init__

Tags: diff tech-debt
description: updated
Revision history for this message
Aaron Bentley (abentley) wrote :

I took a stab at removing the diff algorithm from commit, but it was resistant, because it wants to diff bewteen a tree and its multiple parents.

description: updated
Revision history for this message
Aaron Bentley (abentley) wrote :

InterTree.compare, status, diff and merge all use _iter_changes now.

cmd_renames, cmd_deleted and cmd_added look like they should all be folded into status.

Jelmer Vernooij (jelmer)
tags: added: diff
Jelmer Vernooij (jelmer)
tags: added: check-for-breezy
Jelmer Vernooij (jelmer)
tags: added: tech-debt
removed: check-for-breezy
Changed in brz:
status: New → Triaged
importance: Undecided → Low
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.