Unified Diff Hunk Description Wrong

[RLovelett]

I think there is an issue in the ldiff -u output.

Running ldiff -u data_set_1.csv data_set_2.csv > ldiff.patch generates:

--- spec/data/data_set_1.csv  2013-05-07 15:37:17.592816097 -0500
+++ spec/data/data_set_2.csv    2013-05-07 15:37:17.592816097 -0500
@@ -3,7 +3,7 @@
 3,13
 4,21
 5,31
-6,43
+6,42
 7,57
 8,73
 9,91
@@ -16,7 +16,7 @@
 16,273
 17,307
 18,343
-19,381
+19,200
 20,421
 21,463
 22,507
@@ -28,10 +28,10 @@
 28,813
 29,871
 30,931
-31,993
+31,123
 32,1057
 33,1123
-34,1191
+34,1000
 35,1261
 36,1333
 37,1407
@@ -44,8 +44,9 @@
 44,1981
 45,2071
 46,2163
-47,2257
+47,1524
 48,2353
 49,2451
 50,2500
+51,2520

I believe the last hunk description is wrong. It should be @@ -44,7 +44,8 @@. Running Unix diff confirms my suspicions.

All of the files (including data_set_1.csv and data_set_2.csv) can be found in this Gist.

[halostatue]

Confirmed as still an issue. I'll be looking at it soon.

[halostatue]

Confirmed that this is still an issue even with the #5 and #6 fix that are incoming. The problem is larger than currently reported.

[halostatue]

The release of 1.4 has fixed a number of issues with diff output and the output seems to be better than previously. However, the linked gist no longer exists and I cannot verify that this issue is fixed. I am closing this issue, but if you see any other issues with the output format of diff-lcs not matching diff, please file a new bug.