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 md...@apache.org on 2012/10/12 14:09:40 UTC

svn commit: r1397531 - /jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java

Author: mduerig
Date: Fri Oct 12 12:09:40 2012
New Revision: 1397531

URL: http://svn.apache.org/viewvc?rev=1397531&view=rev
Log:
OAK-350: Unify PropertyState and CoreValue
correct hash code

Modified:
    jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java

Modified: jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java?rev=1397531&r1=1397530&r2=1397531&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java (original)
+++ jackrabbit/oak/trunk/oak-core/src/main/java/org/apache/jackrabbit/oak/value/ValueImpl.java Fri Oct 12 12:09:40 2012
@@ -277,7 +277,12 @@ public class ValueImpl implements Value 
      */
     @Override
     public int hashCode() {
-        return propertyState.hashCode();
+        if (getType() == PropertyType.BINARY) {
+            return propertyState.getValue(Type.BINARY, index).hashCode();
+        }
+        else {
+            return propertyState.getValue(Type.STRING, index).hashCode();
+        }
     }
 
     @Override