MyersDiffer def _find_diagonal had some problems, missing self, and cost parameters.

Review Request #1398 — Created Feb. 9, 2010 and submitted

Information

Review Board

Reviewers

_find_diagonals which is only called if a file has a lot of changes would not work due to missing self, and cost parameters. self should be there since it is in a class, and is called via self._find... And cost is needed for some of the calculations and should have been passed as a parameter.

These changes are needed to allow the diff viewer to work with files with a large amount of change (200+ changes)
Tested a diff with 1280 changes, before the fix it would cause a crash/traceback to line 371, with _find_diagonal takes 11 param, found 12. After this fix, the diff viewer works as expect. Tested with 200+ reviews, and 800+ files, used in our testing system.
chipx86
  1. Thanks. Looks like none of our unit tests ever actually covered this. Any chance you could provide a unit test for this as well?
    
    Committed to master as b916dde.
  2. 
      
Loading...