xmerge: If statement is redundant

Change-Id: I1d4000ae68d109ed01d4197a71f9175f79aa59f0
Reviewed-on: https://gerrit.libreoffice.org/10668
Reviewed-by: Noel Grandin <noelgrandin@gmail.com>
Tested-by: Noel Grandin <noelgrandin@gmail.com>

2 files changed, 3 insertions, 11 deletions

diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/diff/CellNodeIterator.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/diff/CellNodeIterator.java
index 16102e831fc4..c2cad735c5f9 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/diff/CellNodeIterator.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/diff/CellNodeIterator.java
@@ -65,12 +65,8 @@ public final class CellNodeIterator extends NodeIterator  {
 
         // can use an array later to check all possible tags for
         // future expansion
-        if (node.getNodeType() == Node.ELEMENT_NODE &&
-            node.getNodeName().equals(SUPPORTED_TAG1)) {
-            return true;
-        } else {
-            return false;
-        }
+        return node.getNodeType() == Node.ELEMENT_NODE &&
+                node.getNodeName().equals(SUPPORTED_TAG1);
     }
 
 
diff --git a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/DocumentMerge.java b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/DocumentMerge.java
index a4ba311964da..cd939b799ddd 100644
--- a/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/DocumentMerge.java
+++ b/xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/DocumentMerge.java
@@ -67,11 +67,7 @@ public class DocumentMerge implements MergeAlgorithm {
         for (Difference difference : differences) {
             if (difference.getOrgPosition() > currentPosition) {
                 currentPosition = difference.getOrgPosition();
-                if (difference.getOperation() == Difference.DELETE) {
-                    haveDeleteOperation = true;
-                } else  {
-                    haveDeleteOperation = false;
-                }
+                haveDeleteOperation = difference.getOperation() == Difference.DELETE;
             } else if (difference.getOrgPosition() == currentPosition) {
                 if (difference.getOperation() == Difference.DELETE) {
                     haveDeleteOperation = true;