From c70c730b22c847668cf475dc6f841b85297ac3ab Mon Sep 17 00:00:00 2001 From: Sven Gothel Date: Fri, 21 Feb 2014 07:31:12 +0100 Subject: Bitstream: Add static 'long toUint32Long(int)' and 'int toUint32Int(int)' conversion functions --- src/java/com/jogamp/common/util/Bitstream.java | 42 ++++++++++++++++------ .../com/jogamp/common/util/BitstreamData.java | 8 +++-- .../com/jogamp/common/util/TestBitstream00.java | 23 +++++++++++- 3 files changed, 60 insertions(+), 13 deletions(-) (limited to 'src') diff --git a/src/java/com/jogamp/common/util/Bitstream.java b/src/java/com/jogamp/common/util/Bitstream.java index 7bf0c16..419b356 100644 --- a/src/java/com/jogamp/common/util/Bitstream.java +++ b/src/java/com/jogamp/common/util/Bitstream.java @@ -1166,11 +1166,9 @@ public class Bitstream { * Return incoming int32 as read via {@link #readBits31(boolean, int)} * and swap bytes if !bigEndian. *

- * In case the returned value shall be interpreted as unsigned, - * it shall be cast to long as follows: - * 

-     *   final long l = 0xffffffffL & int32;
-     *
+ * In case the returned value shall be interpreted as uint32_t + * utilize {@link #toUint32Long(int)} or {@link #toUint32Int(int)} for + * an appropriate conversion. *

* @param msbFirst if true incoming stream bit order is MSB to LSB, otherwise LSB to MSB. * @param bigEndian if false, swap incoming bytes to little-endian, otherwise leave them as little-endian. @@ -1216,11 +1214,9 @@ public class Bitstream { * Return incoming int32 as read via {@link #readBits31(boolean, int)} * and swap bytes if !bigEndian. *

- * In case the returned value shall be interpreted as unsigned, - * it shall be cast to long as follows: - * 

-     *   final long l = 0xffffffffL & int32;
-     *
+ * In case the returned value shall be interpreted as uint32_t + * utilize {@link #toUint32Long(int)} or {@link #toUint32Int(int)} for + * an appropriate conversion. *

* @param bigEndian if false, swap incoming bytes to little-endian, otherwise leave them as little-endian. * @return the 32bit value, which might be unsigned or 2-complement signed value. @@ -1303,6 +1299,32 @@ public class Bitstream { } } + /** + * Reinterpret the given int32_t value as uint32_t, + * i.e. perform the following cast to long: + * 
+     *   final long l = 0xffffffffL & int32;
+     *
+ */ + public static final long toUint32Long(final int val) { + return 0xffffffffL & val; + } + + /** + * Returns the reinterpreted given int32_t value + * as uint32_t if < {@link Integer#MAX_VALUE} + * as within an int storage. + * Otherwise return -1. + */ + public static final int toUint32Int(final int val) { + final long v = toUint32Long(val); + if( v > Integer.MAX_VALUE ) { + return -1; + } else { + return (int)v; + } + } + public String toString() { return String.format("Bitstream[%s]", toStringImpl()); } diff --git a/src/junit/com/jogamp/common/util/BitstreamData.java b/src/junit/com/jogamp/common/util/BitstreamData.java index 93e8ad0..b5c2c62 100644 --- a/src/junit/com/jogamp/common/util/BitstreamData.java +++ b/src/junit/com/jogamp/common/util/BitstreamData.java @@ -83,6 +83,9 @@ public class BitstreamData { public static String toHexString(int v) { return "0x"+Integer.toHexString(v); } + public static String toHexString(long v) { + return "0x"+Long.toHexString(v); + } public static final String strZeroPadding= "0000000000000000000000000000000000000000000000000000000000000000"; // 64 public static String toBinaryString(int v, int bitCount) { if( 0 == bitCount ) { @@ -113,7 +116,8 @@ public class BitstreamData { return String.format("[%0"+nibbles+"X, %s]", v, toBinaryString(v, bitCount)); } public static String toUnsignedBinaryString(final int int32) { - final long l = 0xffffffffL & int32; - return l+", "+toHexBinaryString(l, 32); + final long l = Bitstream.toUint32Long(int32); + final int i = Bitstream.toUint32Int(int32); + return "(long)"+l+", (int)"+i+", "+toHexBinaryString(l, 32); } } diff --git a/src/junit/com/jogamp/common/util/TestBitstream00.java b/src/junit/com/jogamp/common/util/TestBitstream00.java index ef1fd45..47ed1cf 100644 --- a/src/junit/com/jogamp/common/util/TestBitstream00.java +++ b/src/junit/com/jogamp/common/util/TestBitstream00.java @@ -35,10 +35,13 @@ import java.nio.IntBuffer; import java.nio.LongBuffer; import org.junit.Test; +import org.junit.Assert; import com.jogamp.common.nio.Buffers; import com.jogamp.common.os.Platform; + import static com.jogamp.common.util.BitstreamData.*; + import com.jogamp.junit.util.JunitTracer; import org.junit.FixMethodOrder; @@ -90,7 +93,25 @@ public class TestBitstream00 extends JunitTracer { } @Test - public void test01ShiftSigned() { + public void test01Uint32Conversion() { + testUInt32Conversion(1, 1); + testUInt32Conversion(Integer.MAX_VALUE, Integer.MAX_VALUE); + testUInt32Conversion(0xffff0000, -1); + testUInt32Conversion(0xffffffff, -1); + } + void testUInt32Conversion(final int int32, final int expUint32Int) { + final String int32_hStr = toHexString(int32); + final long l = Bitstream.toUint32Long(int32); + final String l_hStr = toHexString(l); + final int i = Bitstream.toUint32Int(int32); + final String i_hStr = toHexString(i); + System.err.printf("int32_t %012d %10s -> (long) %012d %10s, (int) %012d %10s%n", int32, int32_hStr, l, l_hStr, i, i_hStr); + Assert.assertEquals(int32_hStr, l_hStr); + Assert.assertEquals(expUint32Int, i); + } + + @Test + public void test02ShiftSigned() { shiftSigned(0xA0000000); // negative w/ '1010' top-nibble shiftSigned(-1); } -- cgit v1.2.3