[Libreoffice-commits] core.git: xmerge/source

[rbuj]

 
xmerge/source/xmerge/java/org/openoffice/xmerge/merger/diff/CellNodeIterator.java
 |    8 ++------
 
xmerge/source/xmerge/java/org/openoffice/xmerge/merger/merge/DocumentMerge.java 
  |    6 +-----
 2 files changed, 3 insertions(+), 11 deletions(-)

New commits:
commit 583a35b7872be903520dd9fc342efde23ade929f
Author: rbuj <robert....@gmail.com>
Date:   Thu Jul 31 20:03:30 2014 +0200

    xmerge: If statement is redundant
    
    Change-Id: I1d4000ae68d109ed01d4197a71f9175f79aa59f0
    Reviewed-on: https://gerrit.libreoffice.org/10668
    Reviewed-by: Noel Grandin <noelgran...@gmail.com>
    Tested-by: Noel Grandin <noelgran...@gmail.com>

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 16102e8..c2cad73 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 a4ba311..cd939b7 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;
_______________________________________________
Libreoffice-commits mailing list
libreoffice-comm...@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/libreoffice-commits