Review: First pass at Java JNI interface wrap and andriod toolchain


Malcolm Humphreys <malcolmh...@...>
 

This is mostly about the JNI Java interface, you need to run cmake with (-D OCIO_BUILD_JNIGLUE=TRUE) for the binding to be built. Their is no install target for the JNI build currently I am only running the unit tests with (make JNITests). This has only be run on the OSX so will prob blowup on other platforms.

Any feedback on the Java interface in src/jniglue/org/OpenColorIO would be welcome.

I am having a problem for any JNI experts:

This function sig causes a memory error
Java_org_OpenColorIO_Config_addDisplay(JNIEnv * env, jobject self, jstring display, jstring view, jstring colorSpaceName, jstring looks)

Where this one wouldn't
Java_org_OpenColorIO_Config_addDisplay(JNIEnv * env, jobject self, jstring display, jstring view, jstring colorSpaceName)

Having 4 jstring parameters seems to be the problem, this might just be an issue with the OSX version of Java I'm using I'm not sure. For now Config.addDisplay() is implemented but commented out.

.malcolm

Join ocio-dev@lists.aswf.io to automatically receive all group messages.