Bug 957 - IllegalArgumentException in jocl-demos julia3D
Summary: IllegalArgumentException in jocl-demos julia3D
Status: RESOLVED FIXED
Alias: None
Product: Jocl
Classification: JogAmp
Component: opencl (show other bugs)
Version: 1
Hardware: All all
: --- normal
Assignee: Wade Walker
URL:
Depends on:
Blocks:
 
Reported: 2014-02-01 23:27 CET by Wade Walker
Modified: 2015-07-07 00:53 CEST (History)
1 user (show)

See Also:
Type: ---
SCM Refs:
e935256bf1ad44ceb2d9e3e63810f8d5017733be
Workaround: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Wade Walker 2014-02-01 23:27:43 CET
The julia3d demo fails with an IllegalArgumentException. See http://forum.jogamp.org/jocl-demos-julia3D-fractal-Exception-td3618896.html#a4031423 for the stack trace and suggested fix.
Comment 1 Sven Gothel 2014-02-16 06:46:19 CET
Patrick:
I know this is two years old, but there were no replies, and in case someone else stumbles across this thread, RenderingConfig32.java, RenderingConfig64.java, Vec32.java, and Vec64.java all seem to define their structures incorrectly. Perhaps the way they are supposed to be defined changed.

To fix it, replace each occurrence of setIntAt(x, val) with setIntAt(x*4, val) (e.g. setIntAt(1, val) -> setIntAt(4, val))
Do the same thing with getIntAt(x), getFloatAt(x), setFloatAt(x, val), setFloatsAt(x, val), and getFloatsAt(x).

Hope this helps. 

+++

We need to generate said classes w/ our StructAccessor
either via explicit rules or the annotation - see GlueGen.

While doing so - it would be great to have jocl-demos ant build files to be replaced as well.
Comment 2 Wade Walker 2014-02-16 16:08:51 CET
I plan to take a look at this once I finish fixing the JOCL JUnit tests.
Comment 3 Wade Walker 2015-07-07 00:53:58 CEST
Done. Also fixed RadixSort on OS X, though it still doesn't work right on Windows (at least not on my box).