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 2016/12/14 13:19:42 UTC

svn commit: r1774221 - in /jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment: MapRecord.java SegmentNodeState.java

Author: mduerig
Date: Wed Dec 14 13:19:42 2016
New Revision: 1774221

URL: http://svn.apache.org/viewvc?rev=1774221&view=rev
Log:
OAK-5293: Static code analysis and code cleanup
Remove unnecessary boxing

Modified:
    jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapRecord.java
    jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeState.java

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapRecord.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapRecord.java?rev=1774221&r1=1774220&r2=1774221&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapRecord.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/MapRecord.java Wed Dec 14 13:19:42 2016
@@ -212,7 +212,7 @@ public class MapRecord extends Record {
             assert p <= i && i <= q;
 
             long iH = segment.readInt(getRecordNumber(), 4 + i * 4) & HASH_MASK;
-            int diff = Long.valueOf(iH).compareTo(Long.valueOf(h));
+            int diff = Long.valueOf(iH).compareTo(h);
             if (diff == 0) {
                 RecordId keyId = segment.readRecordId(getRecordNumber(), 4 + size * 4, i * 2);
                 RecordId valueId = segment.readRecordId(getRecordNumber(), 4 + size * 4, i * 2 + 1);

Modified: jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeState.java
URL: http://svn.apache.org/viewvc/jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeState.java?rev=1774221&r1=1774220&r2=1774221&view=diff
==============================================================================
--- jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeState.java (original)
+++ jackrabbit/oak/trunk/oak-segment-tar/src/main/java/org/apache/jackrabbit/oak/segment/SegmentNodeState.java Wed Dec 14 13:19:42 2016
@@ -493,7 +493,7 @@ public class SegmentNodeState extends Re
         while (beforeIndex < beforeProperties.length
                 && afterIndex < afterProperties.length) {
             int d = Integer.valueOf(afterProperties[afterIndex].hashCode())
-                    .compareTo(Integer.valueOf(beforeProperties[beforeIndex].hashCode()));
+                    .compareTo(beforeProperties[beforeIndex].hashCode());
             if (d == 0) {
                 d = afterProperties[afterIndex].getName().compareTo(
                         beforeProperties[beforeIndex].getName());