diff options
| author | Allen Byrne <50328838+byrnHDF@users.noreply.github.com> | 2022-10-03 18:31:32 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-03 18:31:32 (GMT) |
| commit | c114bd64b0662a010b0a9c502b921adfab00f1e6 (patch) | |
| tree | 9bfe24cf7fa1618398e2a7e053483ff0c8043e51 /java/src/jni/h5jni.h | |
| parent | 1fb83da21531da8e531234fa6ab810b7a90d7ee9 (diff) | |
| download | hdf5-c114bd64b0662a010b0a9c502b921adfab00f1e6.zip hdf5-c114bd64b0662a010b0a9c502b921adfab00f1e6.tar.gz hdf5-c114bd64b0662a010b0a9c502b921adfab00f1e6.tar.bz2 | |
Fix mostly java warnings, and cpp cast (#2134)
Diffstat (limited to 'java/src/jni/h5jni.h')
| -rw-r--r-- | java/src/jni/h5jni.h | 17 |
1 files changed, 17 insertions, 0 deletions
diff --git a/java/src/jni/h5jni.h b/java/src/jni/h5jni.h index 2c93846..5af5a91 100644 --- a/java/src/jni/h5jni.h +++ b/java/src/jni/h5jni.h @@ -257,6 +257,23 @@ do { \ (*envptr)->ReleaseStringUTFChars(envptr, pinnedString, stringToRelease); \ } while (0) +/* + * Above String macros may be incorrect, suggested code for getting a cstr from java + * int jstr_to_cstr(JNIEnv *jenv, jstring j_str, char *c_str, size_t cstr_len) + * { + * int32_t j_len, c_len; + * + * c_len = (*jenv)->GetStringUTFLength(jenv, j_str); + * if (c_len > (int32_t)cstr_len) + * return -ENAMETOOLONG; + * j_len = (*jenv)->GetStringLength(jenv, j_str); + * (*jenv)->GetStringUTFRegion(jenv, j_str, 0, j_len, c_str); + * if ((*jenv)->ExceptionCheck(jenv)) + * return -EIO; + * return 0; + * } + * + */ #ifdef __cplusplus extern "C" { |