> > Err, is that really correct? AIUI packages are dot-separated.
Yes, but apparently not for JNI: http://java.sun.com/j2se/1.5.0/docs/guide/jni/spec/functions.html "name: a fully-qualified class name (that is, a package name, delimited by “ /”, followed by the class name)" When you produce patches, please use the -u option. Oops: --- bindings.c 2009-05-27 22:17:32.000000000 +1000 +++ bindings.orig 2009-05-27 22:05:58.000000000 +1000 @@ -66,7 +66,7 @@ char message[lenmsg + 2 + lenerr + 1]; snprintf(message, sizeof(message), "%s: %s", msg, error); - if (!(jcexcep = (*jenv)->FindClass(jenv, "org/a11y/BrlAPI/Error"))) { + if (!(jcexcep = (*jenv)->FindClass(jenv, "org.a11y.BrlAPI.Error"))) { ThrowException(jenv, ERR_NULLPTR, "ThrowBrlapiErrorFindClass"); return; } @@ -97,7 +97,7 @@ } (*env)->SetByteArrayRegion(env, jbuf, 0, size, (jbyte *) buf); - if (!(jcexcep = (*env)->FindClass(env, "org/a11y/BrlAPI/Exception"))) { + if (!(jcexcep = (*env)->FindClass(env, "org.a11y.BrlAPI.Exception"))) { ThrowException(env, ERR_NULLPTR, "exceptionHandlerFindClass"); return; } @@ -130,7 +130,7 @@ jfieldID handleID; \ GET_CLASS(jenv, jcls, jobj, ret); \ GET_ID(jenv, handleID, jcls, "handle", "J", ret); \ - handle = (void*) (intptr_t) (*jenv)->GetLongField(jenv, jobj, handleID); \ + handle = (void*) (intptr_t) (*jenv)->GetLongField(jenv, jcls, handleID); \ if (!handle) { \ ThrowException((jenv), ERR_NULLPTR, "connection has been closed"); \ return ret; \ @@ -155,8 +155,7 @@ ThrowException(jenv, ERR_OUTOFMEM, __func__); return -1; } - - (*jenv)->SetLongField(jenv, jobj, handleID, (jlong) (intptr_t) handle); + (*jenv)->SetLongField(jenv, jcls, handleID, (jlong) (intptr_t) handle); env = jenv; @@ -232,7 +231,7 @@ brlapi__closeConnection(handle); free((void*) (intptr_t) handle); - (*jenv)->SetLongField(jenv, jobj, handleID, (jlong) (intptr_t) NULL); + (*jenv)->SetLongField(jenv, jcls, handleID, (jlong) (intptr_t) NULL); } JNIEXPORT jstring JNICALL Java_org_a11y_BrlAPI_Native_getDriverName(JNIEnv *jenv, jobject jobj) { @@ -264,7 +263,7 @@ return NULL; } - if (!(jcsize = (*jenv)->FindClass(jenv, "org/a11y/BrlAPI/DisplaySize"))) { + if (!(jcsize = (*jenv)->FindClass(jenv, "org.a11y.BrlAPI.DisplaySize"))) { ThrowException(jenv, ERR_NULLPTR, __func__); return NULL; }
_______________________________________________ This message was sent via the BRLTTY mailing list. To post a message, send an e-mail to: [email protected] For general information, go to: http://mielke.cc/mailman/listinfo/brltty
