You are viewing a plain text version of this content. The canonical link for it is here.

Posted to oak-commits@jackrabbit.apache.org by al...@apache.org on 2012/10/11 10:44:26 UTC

svn commit: r1396947 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java

Author: alexparvulescu
Date: Thu Oct 11 08:44:25 2012
New Revision: 1396947

URL: http://svn.apache.org/viewvc?rev=1396947&view=rev
Log:
OAK-350: Unify PropertyState and CoreValue
 - fixed equals in the case of BINARY values

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java?rev=1396947&r1=1396946&r2=1396947&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/plugins/memory/EmptyPropertyState.java Thu Oct 11 08:44:25 2012
@@ -108,7 +108,8 @@ abstract class EmptyPropertyState implem
                 return false;
             }
             if (getType().tag() == PropertyType.BINARY) {
-                return getValue(BINARIES).equals(that.getValue(BINARIES));
+                return Iterables.elementsEqual(
+                        getValue(BINARIES), that.getValue(BINARIES));
             }
             else {
                 return Iterables.elementsEqual(