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());