wumpz · wumpz · commit 06f1e0593efc · 2020-12-13T23:48:24.000+01:00
diff --git a/java-diff-utils/src/main/java/com/github/difflib/patch/AbstractDelta.java b/java-diff-utils/src/main/java/com/github/difflib/patch/AbstractDelta.java
@@ -90,9 +90,6 @@ public boolean equals(Object obj) {
         if (!Objects.equals(this.target, other.target)) {
             return false;
         }
-        if (this.type != other.type) {
-            return false;
-        }
-        return true;
+        return this.type == other.type;
     }
 }
diff --git a/java-diff-utils/src/main/java/com/github/difflib/unifieddiff/UnifiedDiffReader.java b/java-diff-utils/src/main/java/com/github/difflib/unifieddiff/UnifiedDiffReader.java
@@ -91,7 +91,7 @@ private UnifiedDiff parse() throws IOException, UnifiedDiffParserException {
             if (!CHUNK.validLine(line)) {
                 initFileIfNecessary();
                 while (line != null && !CHUNK.validLine(line)) {
-                    if (processLine(line, DIFF_COMMAND, INDEX, FROM_FILE, TO_FILE, NEW_FILE_MODE, DELETED_FILE_MODE) == false) {
+                    if (!processLine(line, DIFF_COMMAND, INDEX, FROM_FILE, TO_FILE, NEW_FILE_MODE, DELETED_FILE_MODE)) {
                         throw new UnifiedDiffParserException("expected file start line not found");
                     }
                     line = READER.readLine();
@@ -100,7 +100,7 @@ private UnifiedDiff parse() throws IOException, UnifiedDiffParserException {
             if (line != null) {
                 processLine(line, CHUNK);
                 while ((line = READER.readLine()) != null) {
-                    if (processLine(line, LINE_NORMAL, LINE_ADD, LINE_DEL) == false) {
+                    if (!processLine(line, LINE_NORMAL, LINE_ADD, LINE_DEL)) {
                         throw new UnifiedDiffParserException("expected data line not found");
                     }
                     if ((originalTxt.size() == old_size && revisedTxt.size() == new_size)

Original file line number	Diff line number	Diff line change
`@@ -90,9 +90,6 @@ public boolean equals(Object obj) {`
`90`	`90`	`if (!Objects.equals(this.target, other.target)) {`
`91`	`91`	`return false;`
`92`	`92`	`}`
`93`		`- if (this.type != other.type) {`
`94`		`- return false;`
`95`		`- }`
`96`		`- return true;`
	`93`	`+ return this.type == other.type;`
`97`	`94`	`}`
`98`	`95`	`}`