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

Posted to commits@directmemory.apache.org by no...@apache.org on 2012/12/16 16:39:00 UTC

svn commit: r1422604 - /directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java

Author: noctarius
Date: Sun Dec 16 15:38:59 2012
New Revision: 1422604

URL: http://svn.apache.org/viewvc?rev=1422604&view=rev
Log:
DIRECTMEMORY-120 Fix ByteOrderUtils::buildLong which uses int values for calculation of the resulting long

Modified:
    directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java

Modified: directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java
URL: http://svn.apache.org/viewvc/directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java?rev=1422604&r1=1422603&r2=1422604&view=diff
==============================================================================
--- directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java (original)
+++ directmemory/trunk/directmemory-cache/src/main/java/org/apache/directmemory/memory/buffer/ByteOrderUtils.java Sun Dec 16 15:38:59 2012
@@ -156,8 +156,8 @@ public class ByteOrderUtils
 
     private static long buildLong( byte b7, byte b6, byte b5, byte b4, byte b3, byte b2, byte b1, byte b0 )
     {
-        return ( ( ( ( b7 & 0xFF ) << 56 ) | ( ( b6 & 0xFF ) << 48 ) | ( ( b5 & 0xFF ) << 40 ) | ( ( b4 & 0xFF ) << 32 )
-            | ( ( b3 & 0xFF ) << 24 ) | ( ( b2 & 0xFF ) << 16 ) | ( ( b1 & 0xFF ) << 8 ) | ( ( b0 & 0xFF ) << 0 ) ) );
+        return ( ( ( ( b7 & 0xFFL ) << 56 ) | ( ( b6 & 0xFFL ) << 48 ) | ( ( b5 & 0xFFL ) << 40 )
+            | ( ( b4 & 0xFFL ) << 32 ) | ( ( b3 & 0xFFL ) << 24 ) | ( ( b2 & 0xFFL ) << 16 ) | ( ( b1 & 0xFFL ) << 8 ) | ( ( b0 & 0xFFL ) << 0 ) ) );
     }
 
 }