summaryrefslogtreecommitdiffstats
path: root/src/gleem/TestTranslate1.java
diff options
context:
space:
mode:
authorSven Gothel <[email protected]>2010-09-24 00:16:24 +0200
committerSven Gothel <[email protected]>2010-09-24 00:16:24 +0200
commitd1830d4ccd8c2db30896e987638228c45e251564 (patch)
treea9faf0015e89f7dc1409fa34be273e747d11f003 /src/gleem/TestTranslate1.java
parent4bb9fed247d8151b317c32dd1f8d7bde03a1bcb6 (diff)
Picking unsigned integer handling fix.
Julien Gouesse send me this patch to the jogamp ML on 2010-09-09 16:24 Thu +0200: Picking.java is a provided example of JOGL 2 here: http://jogamp.org/jogl-demos/src/demos/misc/Picking.java http://jogamp.org/jogl-demos/src/demos/misc/Picking.java The both lines above are wrong: z1 = (float) buffer.get(offset) / 0x7fffffff; offset++; z2 = (float) buffer.get(offset) / 0x7fffffff; offset++; Actually, the select buffer contains 32-bits unsigned integers stored into 32-bits signed integers. Then, the last digit is wrongly used as a sign bit. As Java has no unsigned type, at least 33 bits are required to store these positive values with their sign. Therefore, the long type (64 bits) should be used. The both lines above often work correctly except when the depth values are bigger than 2^31. I assume that they have been copied from an example written in C++. We should rather do this: z1 = (float) (buffer.get(offset)& 0xffffffffL) / 0x7fffffff; offset++; z2 = (float) (buffer.get(offset)& 0xffffffffL) / 0x7fffffff; offset++;
Diffstat (limited to 'src/gleem/TestTranslate1.java')
0 files changed, 0 insertions, 0 deletions