Correct mistaken use of InsertDelta instead of DeleteDelta

catbref · catbref · commit 1d6d7c601699 · 2018-02-28T13:08:12.000Z
Bad copy&paste caused use of InsertDelta instead of DeleteDelta when applying fix for issue dnaumenko#20. Also improved test to cover empty-context patches that delete lines to cover this case.
diff --git a/src/main/java/difflib/DiffUtils.java b/src/main/java/difflib/DiffUtils.java
@@ -214,7 +214,7 @@ public static void processRawChunk(List<String[]> rawChunk, Patch patch, int old
 						old_ln, oldChunkLines), new Chunk<String>(new_ln - 1,
 						newChunkLines)));
 			} else if (newChunkLines.isEmpty()) {
-				patch.addDelta(new InsertDelta<String>(new Chunk<String>(
+				patch.addDelta(new DeleteDelta<String>(new Chunk<String>(
 						old_ln - 1, oldChunkLines), new Chunk<String>(new_ln,
 						newChunkLines)));
 			} else {
diff --git a/src/test/resources/mocks/uc_insert_patch.txt b/src/test/resources/mocks/uc_insert_patch.txt
@@ -2,3 +2,5 @@
 +++ uc_insert_revised.txt	2011-06-14 16:20:37.654820000 +0300
 @@ -2,0 +3 @@
 +not
+@@ -5 +5,0 @@
+-file
diff --git a/src/test/resources/mocks/uc_original.txt b/src/test/resources/mocks/uc_original.txt
@@ -2,3 +2,4 @@ This
 is
 a
 test
+file

Original file line number	Diff line number	Diff line change
`@@ -2,3 +2,4 @@ This`
`2`	`2`	`is`
`3`	`3`	`a`
`4`	`4`	`test`
	`5`	`+file`