[FFmpeg-devel] [PATCH 1/7] avcodec: move ffjni to avutil/jniutils

Matthieu Bouron matthieu.bouron at gmail.com
Thu Feb 15 00:23:21 EET 2024


On Wed, Feb 14, 2024 at 07:18:04PM +0100, Michael Niedermayer wrote:
> On Tue, Feb 13, 2024 at 11:50:09PM +0100, Matthieu Bouron wrote:
> > This will allow to use the jni utils in libavformat. This will be mostly useful
> > to add Android content-uri support.
> > 
> > This deprecates avcodec/jni.h functions in favor of the ones from avutil/jni.h.
> > ---
> >  doc/APIchanges                             |   6 +
> >  libavcodec/Makefile                        |   3 +-
> >  libavcodec/jni.c                           |  48 +----
> >  libavcodec/jni.h                           |   8 +
> >  libavcodec/mediacodec.c                    |   6 +-
> >  libavcodec/mediacodec_surface.c            |   6 +-
> >  libavcodec/mediacodec_wrapper.c            | 200 ++++++++++-----------
> >  libavcodec/mediacodecdec.c                 |   3 +-
> >  libavutil/Makefile                         |   4 +
> >  libavutil/jni.c                            |  78 ++++++++
> >  libavutil/jni.h                            |  46 +++++
> >  libavcodec/ffjni.c => libavutil/jniutils.c |  36 ++--
> >  libavcodec/ffjni.h => libavutil/jniutils.h |  26 ++-
> >  13 files changed, 283 insertions(+), 187 deletions(-)
> >  create mode 100644 libavutil/jni.c
> >  create mode 100644 libavutil/jni.h
> >  rename libavcodec/ffjni.c => libavutil/jniutils.c (88%)
> >  rename libavcodec/ffjni.h => libavutil/jniutils.h (84%)
> 
> this breaks build
> 
>  make
> CC	libavutil/jni.o
> libavutil/jni.c:68:5: error: no previous prototype for ‘av_jni_set_java_vm’ [-Werror=missing-prototypes]
>  int av_jni_set_java_vm(void *vm, void *log_ctx)
>      ^~~~~~~~~~~~~~~~~~
> libavutil/jni.c:73:7: error: no previous prototype for ‘av_jni_get_java_vm’ [-Werror=missing-prototypes]
>  void *av_jni_get_java_vm(void *log_ctx)
>        ^~~~~~~~~~~~~~~~~~
> cc1: some warnings being treated as errors
> ffbuild/common.mak:81: recipe for target 'libavutil/jni.o' failed
> make: *** [libavutil/jni.o] Error 1

Thanks, new patch attached.

[...]
-------------- next part --------------
>From 5b6a127143d8a3d111f7bef1a3e3bc9aab60a8ea Mon Sep 17 00:00:00 2001
From: Matthieu Bouron <matthieu.bouron at gmail.com>
Date: Mon, 12 Feb 2024 18:33:41 +0100
Subject: [PATCH 1/7] avcodec: move ffjni to avutil/jniutils

This will allow to use the jni utils in libavformat. This will be mostly useful
to add Android content-uri support.

This deprecates avcodec/jni.h functions in favor of the ones from avutil/jni.h.
---
 doc/APIchanges                             |   6 +
 libavcodec/Makefile                        |   3 +-
 libavcodec/jni.c                           |  49 +----
 libavcodec/jni.h                           |   8 +
 libavcodec/mediacodec.c                    |   6 +-
 libavcodec/mediacodec_surface.c            |   6 +-
 libavcodec/mediacodec_wrapper.c            | 200 ++++++++++-----------
 libavcodec/mediacodecdec.c                 |   3 +-
 libavutil/Makefile                         |   4 +
 libavutil/jni.c                            |  78 ++++++++
 libavutil/jni.h                            |  46 +++++
 libavcodec/ffjni.c => libavutil/jniutils.c |  36 ++--
 libavcodec/ffjni.h => libavutil/jniutils.h |  26 ++-
 13 files changed, 284 insertions(+), 187 deletions(-)
 create mode 100644 libavutil/jni.c
 create mode 100644 libavutil/jni.h
 rename libavcodec/ffjni.c => libavutil/jniutils.c (88%)
 rename libavcodec/ffjni.h => libavutil/jniutils.h (84%)

diff --git a/doc/APIchanges b/doc/APIchanges
index 221fea30c2..45611ea7ea 100644
--- a/doc/APIchanges
+++ b/doc/APIchanges
@@ -2,6 +2,12 @@ The last version increases of all libraries were on 2023-02-09
 
 API changes, most recent first:
 
+2024-02-xx - xxxxxxxxxx - lavu 58.39.100 - jni.h
+  Add av_jni_set_jvm() and av_jni_get_jvm().
+
+2024-02-xx - xxxxxxxxxx - lavc 60.40.100 - jni.h
+  Deprecate av_jni_set_java_vm() and av_jni_get_java_vm().
+
 2024-02-xx - xxxxxxxxxx - lavu 58.38.100 - channel_layout.h
   Add av_channel_layout_retype().
 
diff --git a/libavcodec/Makefile b/libavcodec/Makefile
index 470d7cb9b1..f8584d8dfd 100644
--- a/libavcodec/Makefile
+++ b/libavcodec/Makefile
@@ -117,7 +117,7 @@ OBJS-$(CONFIG_IIRFILTER)               += iirfilter.o
 OBJS-$(CONFIG_INFLATE_WRAPPER)         += zlib_wrapper.o
 OBJS-$(CONFIG_INTRAX8)                 += intrax8.o intrax8dsp.o msmpeg4data.o
 OBJS-$(CONFIG_IVIDSP)                  += ivi_dsp.o
-OBJS-$(CONFIG_JNI)                     += ffjni.o jni.o
+OBJS-$(CONFIG_JNI)                     += jni.o
 OBJS-$(CONFIG_JPEGTABLES)              += jpegtables.o
 OBJS-$(CONFIG_LCMS2)                   += fflcms2.o
 OBJS-$(CONFIG_LLAUDDSP)                += lossless_audiodsp.o
@@ -1269,7 +1269,6 @@ SKIPHEADERS-$(CONFIG_AMF)              += amfenc.h
 SKIPHEADERS-$(CONFIG_D3D11VA)          += d3d11va.h dxva2_internal.h
 SKIPHEADERS-$(CONFIG_D3D12VA)          += d3d12va_decode.h
 SKIPHEADERS-$(CONFIG_DXVA2)            += dxva2.h dxva2_internal.h
-SKIPHEADERS-$(CONFIG_JNI)              += ffjni.h
 SKIPHEADERS-$(CONFIG_LCMS2)            += fflcms2.h
 SKIPHEADERS-$(CONFIG_LIBAOM)           += libaom.h
 SKIPHEADERS-$(CONFIG_LIBJXL)           += libjxl.h
diff --git a/libavcodec/jni.c b/libavcodec/jni.c
index ae6490de9d..f301307202 100644
--- a/libavcodec/jni.c
+++ b/libavcodec/jni.c
@@ -20,60 +20,19 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#include "config.h"
-
 #include <stdlib.h>
 
 #include "libavutil/error.h"
-#include "jni.h"
-
-#if CONFIG_JNI
-#include <jni.h>
-#include <pthread.h>
-
-#include "libavutil/log.h"
-#include "ffjni.h"
-
-static void *java_vm;
-static pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;
+#include "libavutil/jni.h"
 
-int av_jni_set_java_vm(void *vm, void *log_ctx)
-{
-    int ret = 0;
-
-    pthread_mutex_lock(&lock);
-    if (java_vm == NULL) {
-        java_vm = vm;
-    } else if (java_vm != vm) {
-        ret = AVERROR(EINVAL);
-        av_log(log_ctx, AV_LOG_ERROR, "A Java virtual machine has already been set");
-    }
-    pthread_mutex_unlock(&lock);
-
-    return ret;
-}
-
-void *av_jni_get_java_vm(void *log_ctx)
-{
-    void *vm;
-
-    pthread_mutex_lock(&lock);
-    vm = java_vm;
-    pthread_mutex_unlock(&lock);
-
-    return vm;
-}
-
-#else
+#include "jni.h"
 
 int av_jni_set_java_vm(void *vm, void *log_ctx)
 {
-    return AVERROR(ENOSYS);
+    return av_jni_set_jvm(vm, log_ctx);
 }
 
 void *av_jni_get_java_vm(void *log_ctx)
 {
-    return NULL;
+    return av_jni_get_jvm(log_ctx);
 }
-
-#endif
diff --git a/libavcodec/jni.h b/libavcodec/jni.h
index dd99e92611..49ddab4120 100644
--- a/libavcodec/jni.h
+++ b/libavcodec/jni.h
@@ -23,6 +23,8 @@
 #ifndef AVCODEC_JNI_H
 #define AVCODEC_JNI_H
 
+#include <libavutil/attributes.h>
+
 /*
  * Manually set a Java virtual machine which will be used to retrieve the JNI
  * environment. Once a Java VM is set it cannot be changed afterwards, meaning
@@ -32,7 +34,10 @@
  * @param vm Java virtual machine
  * @param log_ctx context used for logging, can be NULL
  * @return 0 on success, < 0 otherwise
+ *
+ * @deprecated use av_jni_set_jvm from libavutil/jni.h
  */
+attribute_deprecated
 int av_jni_set_java_vm(void *vm, void *log_ctx);
 
 /*
@@ -40,7 +45,10 @@ int av_jni_set_java_vm(void *vm, void *log_ctx);
  *
  * @param vm Java virtual machine
  * @return a pointer to the Java virtual machine
+ *
+ * @deprecated use av_jni_get_jvm from libavutil/jni.h
  */
+attribute_deprecated
 void *av_jni_get_java_vm(void *log_ctx);
 
 #endif /* AVCODEC_JNI_H */
diff --git a/libavcodec/mediacodec.c b/libavcodec/mediacodec.c
index 33bde8112e..2f1ccaa480 100644
--- a/libavcodec/mediacodec.c
+++ b/libavcodec/mediacodec.c
@@ -31,9 +31,9 @@
 #include <jni.h>
 
 #include "libavcodec/avcodec.h"
+#include "libavutil/jniutils.h"
 #include "libavutil/mem.h"
 
-#include "ffjni.h"
 #include "mediacodecdec_common.h"
 
 AVMediaCodecContext *av_mediacodec_alloc_context(void)
@@ -46,7 +46,7 @@ int av_mediacodec_default_init(AVCodecContext *avctx, AVMediaCodecContext *ctx,
     int ret = 0;
     JNIEnv *env = NULL;
 
-    env = ff_jni_get_env(avctx);
+    env = avpriv_jni_get_env(avctx);
     if (!env) {
         return AVERROR_EXTERNAL;
     }
@@ -72,7 +72,7 @@ void av_mediacodec_default_free(AVCodecContext *avctx)
         return;
     }
 
-    env = ff_jni_get_env(avctx);
+    env = avpriv_jni_get_env(avctx);
     if (!env) {
         return;
     }
diff --git a/libavcodec/mediacodec_surface.c b/libavcodec/mediacodec_surface.c
index ef41cdafa7..748f3e2804 100644
--- a/libavcodec/mediacodec_surface.c
+++ b/libavcodec/mediacodec_surface.c
@@ -24,7 +24,7 @@
 #include <jni.h>
 
 #include "libavutil/mem.h"
-#include "ffjni.h"
+#include "libavutil/jniutils.h"
 #include "mediacodec_surface.h"
 
 FFANativeWindow *ff_mediacodec_surface_ref(void *surface, void *native_window, void *log_ctx)
@@ -38,7 +38,7 @@ FFANativeWindow *ff_mediacodec_surface_ref(void *surface, void *native_window, v
     if (surface) {
         JNIEnv *env = NULL;
 
-        env = ff_jni_get_env(log_ctx);
+        env = avpriv_jni_get_env(log_ctx);
         if (env)
             ret->surface = (*env)->NewGlobalRef(env, surface);
     }
@@ -64,7 +64,7 @@ int ff_mediacodec_surface_unref(FFANativeWindow *window, void *log_ctx)
     if (window->surface) {
         JNIEnv *env = NULL;
 
-        env = ff_jni_get_env(log_ctx);
+        env = avpriv_jni_get_env(log_ctx);
         if (env)
             (*env)->DeleteGlobalRef(env, window->surface);
     }
diff --git a/libavcodec/mediacodec_wrapper.c b/libavcodec/mediacodec_wrapper.c
index 0880ddd3ef..bc3eb0a80d 100644
--- a/libavcodec/mediacodec_wrapper.c
+++ b/libavcodec/mediacodec_wrapper.c
@@ -30,9 +30,9 @@
 #include "libavutil/avassert.h"
 #include "libavutil/mem.h"
 #include "libavutil/avstring.h"
+#include "libavutil/jniutils.h"
 
 #include "avcodec.h"
-#include "ffjni.h"
 #include "mediacodec_wrapper.h"
 
 struct JNIAMediaCodecListFields {
@@ -287,14 +287,14 @@ typedef struct FFAMediaCodecJni {
 static const FFAMediaCodec media_codec_jni;
 
 #define JNI_GET_ENV_OR_RETURN(env, log_ctx, ret) do {              \
-    (env) = ff_jni_get_env(log_ctx);                               \
+    (env) = avpriv_jni_get_env(log_ctx);                           \
     if (!(env)) {                                                  \
         return ret;                                                \
     }                                                              \
 } while (0)
 
 #define JNI_GET_ENV_OR_RETURN_VOID(env, log_ctx) do {              \
-    (env) = ff_jni_get_env(log_ctx);                               \
+    (env) = avpriv_jni_get_env(log_ctx);                           \
     if (!(env)) {                                                  \
         return;                                                    \
     }                                                              \
@@ -485,16 +485,16 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
 
     JNI_GET_ENV_OR_RETURN(env, log_ctx, NULL);
 
-    if ((ret = ff_jni_init_jfields(env, &jfields, jni_amediacodeclist_mapping, 0, log_ctx)) < 0) {
+    if ((ret = avpriv_jni_init_jfields(env, &jfields, jni_amediacodeclist_mapping, 0, log_ctx)) < 0) {
         goto done;
     }
 
-    if ((ret = ff_jni_init_jfields(env, &mediaformat_jfields, jni_amediaformat_mapping, 0, log_ctx)) < 0) {
+    if ((ret = avpriv_jni_init_jfields(env, &mediaformat_jfields, jni_amediaformat_mapping, 0, log_ctx)) < 0) {
         goto done;
     }
 
     codec_count = (*env)->CallStaticIntMethod(env, jfields.mediacodec_list_class, jfields.get_codec_count_id);
-    if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+    if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
         goto done;
     }
 
@@ -504,17 +504,17 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
         int is_encoder;
 
         info = (*env)->CallStaticObjectMethod(env, jfields.mediacodec_list_class, jfields.get_codec_info_at_id, i);
-        if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+        if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
             goto done;
         }
 
         types = (*env)->CallObjectMethod(env, info, jfields.get_supported_types_id);
-        if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+        if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
             goto done;
         }
 
         is_encoder = (*env)->CallBooleanMethod(env, info, jfields.is_encoder_id);
-        if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+        if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
             goto done;
         }
 
@@ -524,7 +524,7 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
 
         if (jfields.is_software_only_id) {
             int is_software_only = (*env)->CallBooleanMethod(env, info, jfields.is_software_only_id);
-            if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+            if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                 goto done;
             }
 
@@ -534,11 +534,11 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
         }
 
         codec_name = (*env)->CallObjectMethod(env, info, jfields.get_name_id);
-        if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+        if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
             goto done;
         }
 
-        name = ff_jni_jstring_to_utf_chars(env, codec_name, log_ctx);
+        name = avpriv_jni_jstring_to_utf_chars(env, codec_name, log_ctx);
         if (!name) {
             goto done;
         }
@@ -563,11 +563,11 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
             int profile_count;
 
             type = (*env)->GetObjectArrayElement(env, types, j);
-            if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+            if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                 goto done;
             }
 
-            supported_type = ff_jni_jstring_to_utf_chars(env, type, log_ctx);
+            supported_type = avpriv_jni_jstring_to_utf_chars(env, type, log_ctx);
             if (!supported_type) {
                 goto done;
             }
@@ -577,12 +577,12 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
             }
 
             capabilities = (*env)->CallObjectMethod(env, info, jfields.get_codec_capabilities_id, type);
-            if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+            if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                 goto done;
             }
 
             profile_levels = (*env)->GetObjectField(env, capabilities, jfields.profile_levels_id);
-            if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+            if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                 goto done;
             }
 
@@ -599,12 +599,12 @@ char *ff_AMediaCodecList_getCodecNameByType(const char *mime, int profile, int e
                 }
 
                 profile_level = (*env)->GetObjectArrayElement(env, profile_levels, k);
-                if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+                if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                     goto done;
                 }
 
                 supported_profile = (*env)->GetIntField(env, profile_level, jfields.profile_id);
-                if (ff_jni_exception_check(env, 1, log_ctx) < 0) {
+                if (avpriv_jni_exception_check(env, 1, log_ctx) < 0) {
                     goto done;
                 }
 
@@ -692,8 +692,8 @@ done:
 
     av_freep(&supported_type);
 
-    ff_jni_reset_jfields(env, &jfields, jni_amediacodeclist_mapping, 0, log_ctx);
-    ff_jni_reset_jfields(env, &mediaformat_jfields, jni_amediaformat_mapping, 0, log_ctx);
+    avpriv_jni_reset_jfields(env, &jfields, jni_amediacodeclist_mapping, 0, log_ctx);
+    avpriv_jni_reset_jfields(env, &mediaformat_jfields, jni_amediaformat_mapping, 0, log_ctx);
 
     if (!found_codec) {
         av_freep(&name);
@@ -714,13 +714,13 @@ static FFAMediaFormat *mediaformat_jni_new(void)
     }
     format->api = media_format_jni;
 
-    env = ff_jni_get_env(format);
+    env = avpriv_jni_get_env(format);
     if (!env) {
         av_freep(&format);
         return NULL;
     }
 
-    if (ff_jni_init_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format) < 0) {
+    if (avpriv_jni_init_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format) < 0) {
         goto fail;
     }
 
@@ -740,7 +740,7 @@ fail:
     }
 
     if (!format->object) {
-        ff_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
+        avpriv_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
         av_freep(&format);
     }
 
@@ -758,13 +758,13 @@ static FFAMediaFormat *mediaformat_jni_newFromObject(void *object)
     }
     format->api = media_format_jni;
 
-    env = ff_jni_get_env(format);
+    env = avpriv_jni_get_env(format);
     if (!env) {
         av_freep(&format);
         return NULL;
     }
 
-    if (ff_jni_init_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format) < 0) {
+    if (avpriv_jni_init_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format) < 0) {
         goto fail;
     }
 
@@ -775,7 +775,7 @@ static FFAMediaFormat *mediaformat_jni_newFromObject(void *object)
 
     return (FFAMediaFormat *)format;
 fail:
-    ff_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
+    avpriv_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
 
     av_freep(&format);
 
@@ -797,7 +797,7 @@ static int mediaformat_jni_delete(FFAMediaFormat* ctx)
     (*env)->DeleteGlobalRef(env, format->object);
     format->object = NULL;
 
-    ff_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
+    avpriv_jni_reset_jfields(env, &format->jfields, jni_amediaformat_mapping, 1, format);
 
     av_freep(&format);
 
@@ -816,11 +816,11 @@ static char* mediaformat_jni_toString(FFAMediaFormat* ctx)
     JNI_GET_ENV_OR_RETURN(env, format, NULL);
 
     description = (*env)->CallObjectMethod(env, format->object, format->jfields.to_string_id);
-    if (ff_jni_exception_check(env, 1, NULL) < 0) {
+    if (avpriv_jni_exception_check(env, 1, NULL) < 0) {
         goto fail;
     }
 
-    ret = ff_jni_jstring_to_utf_chars(env, description, format);
+    ret = avpriv_jni_jstring_to_utf_chars(env, description, format);
 fail:
     if (description) {
         (*env)->DeleteLocalRef(env, description);
@@ -841,20 +841,20 @@ static int mediaformat_jni_getInt32(FFAMediaFormat* ctx, const char *name, int32
 
     JNI_GET_ENV_OR_RETURN(env, format, 0);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         ret = 0;
         goto fail;
     }
 
     contains_key = (*env)->CallBooleanMethod(env, format->object, format->jfields.contains_key_id, key);
-    if (!contains_key || (ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if (!contains_key || (ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
     *out = (*env)->CallIntMethod(env, format->object, format->jfields.get_integer_id, key);
-    if ((ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
@@ -880,20 +880,20 @@ static int mediaformat_jni_getInt64(FFAMediaFormat* ctx, const char *name, int64
 
     JNI_GET_ENV_OR_RETURN(env, format, 0);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         ret = 0;
         goto fail;
     }
 
     contains_key = (*env)->CallBooleanMethod(env, format->object, format->jfields.contains_key_id, key);
-    if (!contains_key || (ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if (!contains_key || (ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
     *out = (*env)->CallLongMethod(env, format->object, format->jfields.get_long_id, key);
-    if ((ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
@@ -919,20 +919,20 @@ static int mediaformat_jni_getFloat(FFAMediaFormat* ctx, const char *name, float
 
     JNI_GET_ENV_OR_RETURN(env, format, 0);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         ret = 0;
         goto fail;
     }
 
     contains_key = (*env)->CallBooleanMethod(env, format->object, format->jfields.contains_key_id, key);
-    if (!contains_key || (ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if (!contains_key || (ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
     *out = (*env)->CallFloatMethod(env, format->object, format->jfields.get_float_id, key);
-    if ((ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
@@ -959,20 +959,20 @@ static int mediaformat_jni_getBuffer(FFAMediaFormat* ctx, const char *name, void
 
     JNI_GET_ENV_OR_RETURN(env, format, 0);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         ret = 0;
         goto fail;
     }
 
     contains_key = (*env)->CallBooleanMethod(env, format->object, format->jfields.contains_key_id, key);
-    if (!contains_key || (ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if (!contains_key || (ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
     result = (*env)->CallObjectMethod(env, format->object, format->jfields.get_bytebuffer_id, key);
-    if ((ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
@@ -1017,25 +1017,25 @@ static int mediaformat_jni_getString(FFAMediaFormat* ctx, const char *name, cons
 
     JNI_GET_ENV_OR_RETURN(env, format, 0);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         ret = 0;
         goto fail;
     }
 
     contains_key = (*env)->CallBooleanMethod(env, format->object, format->jfields.contains_key_id, key);
-    if (!contains_key || (ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if (!contains_key || (ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
     result = (*env)->CallObjectMethod(env, format->object, format->jfields.get_string_id, key);
-    if ((ret = ff_jni_exception_check(env, 1, format)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, format)) < 0) {
         ret = 0;
         goto fail;
     }
 
-    *out = ff_jni_jstring_to_utf_chars(env, result, format);
+    *out = avpriv_jni_jstring_to_utf_chars(env, result, format);
     if (!*out) {
         ret = 0;
         goto fail;
@@ -1064,13 +1064,13 @@ static void mediaformat_jni_setInt32(FFAMediaFormat* ctx, const char* name, int3
 
     JNI_GET_ENV_OR_RETURN_VOID(env, format);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         goto fail;
     }
 
     (*env)->CallVoidMethod(env, format->object, format->jfields.set_integer_id, key, value);
-    if (ff_jni_exception_check(env, 1, format) < 0) {
+    if (avpriv_jni_exception_check(env, 1, format) < 0) {
         goto fail;
     }
 
@@ -1090,13 +1090,13 @@ static void mediaformat_jni_setInt64(FFAMediaFormat* ctx, const char* name, int6
 
     JNI_GET_ENV_OR_RETURN_VOID(env, format);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         goto fail;
     }
 
     (*env)->CallVoidMethod(env, format->object, format->jfields.set_long_id, key, value);
-    if (ff_jni_exception_check(env, 1, format) < 0) {
+    if (avpriv_jni_exception_check(env, 1, format) < 0) {
         goto fail;
     }
 
@@ -1116,13 +1116,13 @@ static void mediaformat_jni_setFloat(FFAMediaFormat* ctx, const char* name, floa
 
     JNI_GET_ENV_OR_RETURN_VOID(env, format);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         goto fail;
     }
 
     (*env)->CallVoidMethod(env, format->object, format->jfields.set_float_id, key, value);
-    if (ff_jni_exception_check(env, 1, format) < 0) {
+    if (avpriv_jni_exception_check(env, 1, format) < 0) {
         goto fail;
     }
 
@@ -1143,18 +1143,18 @@ static void mediaformat_jni_setString(FFAMediaFormat* ctx, const char* name, con
 
     JNI_GET_ENV_OR_RETURN_VOID(env, format);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         goto fail;
     }
 
-    string = ff_jni_utf_chars_to_jstring(env, value, format);
+    string = avpriv_jni_utf_chars_to_jstring(env, value, format);
     if (!string) {
         goto fail;
     }
 
     (*env)->CallVoidMethod(env, format->object, format->jfields.set_string_id, key, string);
-    if (ff_jni_exception_check(env, 1, format) < 0) {
+    if (avpriv_jni_exception_check(env, 1, format) < 0) {
         goto fail;
     }
 
@@ -1180,7 +1180,7 @@ static void mediaformat_jni_setBuffer(FFAMediaFormat* ctx, const char* name, voi
 
     JNI_GET_ENV_OR_RETURN_VOID(env, format);
 
-    key = ff_jni_utf_chars_to_jstring(env, name, format);
+    key = avpriv_jni_utf_chars_to_jstring(env, name, format);
     if (!key) {
         goto fail;
     }
@@ -1202,7 +1202,7 @@ static void mediaformat_jni_setBuffer(FFAMediaFormat* ctx, const char* name, voi
     }
 
     (*env)->CallVoidMethod(env, format->object, format->jfields.set_bytebuffer_id, key, buffer);
-    if (ff_jni_exception_check(env, 1, format) < 0) {
+    if (avpriv_jni_exception_check(env, 1, format) < 0) {
         goto fail;
     }
 
@@ -1224,44 +1224,44 @@ static int codec_init_static_fields(FFAMediaCodecJni *codec)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     codec->INFO_TRY_AGAIN_LATER = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.info_try_again_later_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     codec->BUFFER_FLAG_CODEC_CONFIG = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.buffer_flag_codec_config_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     codec->BUFFER_FLAG_END_OF_STREAM = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.buffer_flag_end_of_stream_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     if (codec->jfields.buffer_flag_key_frame_id) {
         codec->BUFFER_FLAG_KEY_FRAME = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.buffer_flag_key_frame_id);
-        if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+        if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
             goto fail;
         }
     }
 
     codec->CONFIGURE_FLAG_ENCODE = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.configure_flag_encode_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     codec->INFO_TRY_AGAIN_LATER = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.info_try_again_later_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     codec->INFO_OUTPUT_BUFFERS_CHANGED = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.info_output_buffers_changed_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
     codec->INFO_OUTPUT_FORMAT_CHANGED = (*env)->GetStaticIntField(env, codec->jfields.mediacodec_class, codec->jfields.info_output_format_changed_id);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         goto fail;
     }
 
@@ -1290,17 +1290,17 @@ static inline FFAMediaCodec *codec_create(int method, const char *arg)
     }
     codec->api = media_codec_jni;
 
-    env = ff_jni_get_env(codec);
+    env = avpriv_jni_get_env(codec);
     if (!env) {
         av_freep(&codec);
         return NULL;
     }
 
-    if (ff_jni_init_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec) < 0) {
+    if (avpriv_jni_init_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec) < 0) {
         goto fail;
     }
 
-    jarg = ff_jni_utf_chars_to_jstring(env, arg, codec);
+    jarg = avpriv_jni_utf_chars_to_jstring(env, arg, codec);
     if (!jarg) {
         goto fail;
     }
@@ -1317,7 +1317,7 @@ static inline FFAMediaCodec *codec_create(int method, const char *arg)
                                             codec->jfields.mediacodec_class,
                                             create_id,
                                             jarg);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         goto fail;
     }
 
@@ -1335,7 +1335,7 @@ static inline FFAMediaCodec *codec_create(int method, const char *arg)
     }
 
     buffer_info = (*env)->NewObject(env, codec->jfields.mediainfo_class, codec->jfields.init_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         goto fail;
     }
 
@@ -1367,7 +1367,7 @@ fail:
             (*env)->DeleteGlobalRef(env, codec->buffer_info);
         }
 
-        ff_jni_reset_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec);
+        avpriv_jni_reset_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec);
         av_freep(&codec);
     }
 
@@ -1397,7 +1397,7 @@ static int mediacodec_jni_delete(FFAMediaCodec* ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.release_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
     }
 
@@ -1413,7 +1413,7 @@ static int mediacodec_jni_delete(FFAMediaCodec* ctx)
     (*env)->DeleteGlobalRef(env, codec->buffer_info);
     codec->buffer_info = NULL;
 
-    ff_jni_reset_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec);
+    avpriv_jni_reset_jfields(env, &codec->jfields, jni_amediacodec_mapping, 1, codec);
 
     av_freep(&codec);
 
@@ -1430,11 +1430,11 @@ static char *mediacodec_jni_getName(FFAMediaCodec *ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, NULL);
 
     name = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_name_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         goto fail;
     }
 
-    ret = ff_jni_jstring_to_utf_chars(env, name, codec);
+    ret = avpriv_jni_jstring_to_utf_chars(env, name, codec);
 
 fail:
     if (name) {
@@ -1465,20 +1465,20 @@ static int mediacodec_jni_configure(FFAMediaCodec *ctx,
         }
 
         (*env)->CallVoidMethod(env, codec->object, codec->jfields.configure_id, format->object, NULL, NULL, flags);
-        if (ff_jni_exception_check(env, 1, codec) < 0)
+        if (avpriv_jni_exception_check(env, 1, codec) < 0)
             return AVERROR_EXTERNAL;
 
         if (!surface)
             return 0;
 
         (*env)->CallVoidMethod(env, codec->object, codec->jfields.set_input_surface_id, surface);
-        if (ff_jni_exception_check(env, 1, codec) < 0)
+        if (avpriv_jni_exception_check(env, 1, codec) < 0)
             return AVERROR_EXTERNAL;
         return 0;
     } else {
         (*env)->CallVoidMethod(env, codec->object, codec->jfields.configure_id, format->object, surface, NULL, flags);
     }
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1496,7 +1496,7 @@ static int mediacodec_jni_start(FFAMediaCodec* ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.start_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1514,7 +1514,7 @@ static int mediacodec_jni_stop(FFAMediaCodec* ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.stop_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1532,7 +1532,7 @@ static int mediacodec_jni_flush(FFAMediaCodec* ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.flush_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1550,7 +1550,7 @@ static int mediacodec_jni_releaseOutputBuffer(FFAMediaCodec* ctx, size_t idx, in
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.release_output_buffer_id, (jint)idx, (jboolean)render);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1568,7 +1568,7 @@ static int mediacodec_jni_releaseOutputBufferAtTime(FFAMediaCodec *ctx, size_t i
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.release_output_buffer_at_time_id, (jint)idx, (jlong)timestampNs);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1586,7 +1586,7 @@ static ssize_t mediacodec_jni_dequeueInputBuffer(FFAMediaCodec* ctx, int64_t tim
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     ret = (*env)->CallIntMethod(env, codec->object, codec->jfields.dequeue_input_buffer_id, timeoutUs);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1604,7 +1604,7 @@ static int mediacodec_jni_queueInputBuffer(FFAMediaCodec* ctx, size_t idx, off_t
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.queue_input_buffer_id, (jint)idx, (jint)offset, (jint)size, time, flags);
-    if ((ret = ff_jni_exception_check(env, 1, codec)) < 0) {
+    if ((ret = avpriv_jni_exception_check(env, 1, codec)) < 0) {
         ret = AVERROR_EXTERNAL;
         goto fail;
     }
@@ -1622,27 +1622,27 @@ static ssize_t mediacodec_jni_dequeueOutputBuffer(FFAMediaCodec* ctx, FFAMediaCo
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     ret = (*env)->CallIntMethod(env, codec->object, codec->jfields.dequeue_output_buffer_id, codec->buffer_info, timeoutUs);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
     info->flags = (*env)->GetIntField(env, codec->buffer_info, codec->jfields.flags_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
     info->offset = (*env)->GetIntField(env, codec->buffer_info, codec->jfields.offset_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
     info->presentationTimeUs = (*env)->GetLongField(env, codec->buffer_info, codec->jfields.presentation_time_us_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
     info->size = (*env)->GetIntField(env, codec->buffer_info, codec->jfields.size_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
@@ -1661,24 +1661,24 @@ static uint8_t* mediacodec_jni_getInputBuffer(FFAMediaCodec* ctx, size_t idx, si
 
     if (codec->has_get_i_o_buffer) {
         buffer = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_input_buffer_id, (jint)idx);
-        if (ff_jni_exception_check(env, 1, codec) < 0) {
+        if (avpriv_jni_exception_check(env, 1, codec) < 0) {
             goto fail;
         }
     } else {
         if (!codec->input_buffers) {
             input_buffers = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_input_buffers_id);
-            if (ff_jni_exception_check(env, 1, codec) < 0) {
+            if (avpriv_jni_exception_check(env, 1, codec) < 0) {
                 goto fail;
             }
 
             codec->input_buffers = (*env)->NewGlobalRef(env, input_buffers);
-            if (ff_jni_exception_check(env, 1, codec) < 0) {
+            if (avpriv_jni_exception_check(env, 1, codec) < 0) {
                 goto fail;
             }
         }
 
         buffer = (*env)->GetObjectArrayElement(env, codec->input_buffers, idx);
-        if (ff_jni_exception_check(env, 1, codec) < 0) {
+        if (avpriv_jni_exception_check(env, 1, codec) < 0) {
             goto fail;
         }
     }
@@ -1709,24 +1709,24 @@ static uint8_t* mediacodec_jni_getOutputBuffer(FFAMediaCodec* ctx, size_t idx, s
 
     if (codec->has_get_i_o_buffer) {
         buffer = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_output_buffer_id, (jint)idx);
-        if (ff_jni_exception_check(env, 1, codec) < 0) {
+        if (avpriv_jni_exception_check(env, 1, codec) < 0) {
             goto fail;
         }
     } else {
         if (!codec->output_buffers) {
             output_buffers = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_output_buffers_id);
-            if (ff_jni_exception_check(env, 1, codec) < 0) {
+            if (avpriv_jni_exception_check(env, 1, codec) < 0) {
                 goto fail;
             }
 
             codec->output_buffers = (*env)->NewGlobalRef(env, output_buffers);
-            if (ff_jni_exception_check(env, 1, codec) < 0) {
+            if (avpriv_jni_exception_check(env, 1, codec) < 0) {
                 goto fail;
             }
         }
 
         buffer = (*env)->GetObjectArrayElement(env, codec->output_buffers, idx);
-        if (ff_jni_exception_check(env, 1, codec) < 0) {
+        if (avpriv_jni_exception_check(env, 1, codec) < 0) {
             goto fail;
         }
     }
@@ -1756,7 +1756,7 @@ static FFAMediaFormat* mediacodec_jni_getOutputFormat(FFAMediaCodec* ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, NULL);
 
     mediaformat = (*env)->CallObjectMethod(env, codec->object, codec->jfields.get_output_format_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         goto fail;
     }
 
@@ -1820,7 +1820,7 @@ static int mediacodec_jni_cleanOutputBuffers(FFAMediaCodec *ctx)
         if (codec->output_buffers) {
             JNIEnv *env = NULL;
 
-            env = ff_jni_get_env(codec);
+            env = avpriv_jni_get_env(codec);
             if (!env) {
                 ret = AVERROR_EXTERNAL;
                 goto fail;
@@ -1843,7 +1843,7 @@ static int mediacodec_jni_signalEndOfInputStream(FFAMediaCodec *ctx)
     JNI_GET_ENV_OR_RETURN(env, codec, AVERROR_EXTERNAL);
 
     (*env)->CallVoidMethod(env, codec->object, codec->jfields.signal_end_of_input_stream_id);
-    if (ff_jni_exception_check(env, 1, codec) < 0) {
+    if (avpriv_jni_exception_check(env, 1, codec) < 0) {
         return AVERROR_EXTERNAL;
     }
 
diff --git a/libavcodec/mediacodecdec.c b/libavcodec/mediacodecdec.c
index b8587289a2..128a5bfef1 100644
--- a/libavcodec/mediacodecdec.c
+++ b/libavcodec/mediacodecdec.c
@@ -29,6 +29,7 @@
 #include "libavutil/common.h"
 #include "libavutil/opt.h"
 #include "libavutil/intreadwrite.h"
+#include "libavutil/jni.h"
 #include "libavutil/pixfmt.h"
 #include "libavutil/internal.h"
 
@@ -314,7 +315,7 @@ static av_cold int mediacodec_decode_init(AVCodecContext *avctx)
     MediaCodecH264DecContext *s = avctx->priv_data;
 
     if (s->use_ndk_codec < 0)
-        s->use_ndk_codec = !av_jni_get_java_vm(avctx);
+        s->use_ndk_codec = !av_jni_get_jvm(avctx);
 
     format = ff_AMediaFormat_new(s->use_ndk_codec);
     if (!format) {
diff --git a/libavutil/Makefile b/libavutil/Makefile
index e7709b97d0..e28944b1b6 100644
--- a/libavutil/Makefile
+++ b/libavutil/Makefile
@@ -56,6 +56,7 @@ HEADERS = adler32.h                                                     \
           imgutils.h                                                    \
           intfloat.h                                                    \
           intreadwrite.h                                                \
+          jni.h                                                         \
           lfg.h                                                         \
           log.h                                                         \
           lzo.h                                                         \
@@ -146,6 +147,7 @@ OBJS = adler32.o                                                        \
        imgutils.o                                                       \
        integer.o                                                        \
        intmath.o                                                        \
+       jni.o                                                            \
        lfg.o                                                            \
        lls.o                                                            \
        log.o                                                            \
@@ -195,6 +197,7 @@ OBJS-$(CONFIG_CUDA)                     += hwcontext_cuda.o
 OBJS-$(CONFIG_D3D11VA)                  += hwcontext_d3d11va.o
 OBJS-$(CONFIG_D3D12VA)                  += hwcontext_d3d12va.o
 OBJS-$(CONFIG_DXVA2)                    += hwcontext_dxva2.o
+OBJS-$(CONFIG_JNI)                      += jniutils.o
 OBJS-$(CONFIG_LIBDRM)                   += hwcontext_drm.o
 OBJS-$(CONFIG_MACOS_KPERF)              += macos_kperf.o
 OBJS-$(CONFIG_MEDIACODEC)               += hwcontext_mediacodec.o
@@ -219,6 +222,7 @@ SKIPHEADERS-$(CONFIG_CUDA)             += hwcontext_cuda_internal.h     \
 SKIPHEADERS-$(CONFIG_D3D11VA)          += hwcontext_d3d11va.h
 SKIPHEADERS-$(CONFIG_D3D12VA)          += hwcontext_d3d12va.h
 SKIPHEADERS-$(CONFIG_DXVA2)            += hwcontext_dxva2.h
+SKIPHEADERS-$(CONFIG_JNI)              += jniutils.h
 SKIPHEADERS-$(CONFIG_QSV)              += hwcontext_qsv.h
 SKIPHEADERS-$(CONFIG_OPENCL)           += hwcontext_opencl.h
 SKIPHEADERS-$(CONFIG_VAAPI)            += hwcontext_vaapi.h
diff --git a/libavutil/jni.c b/libavutil/jni.c
new file mode 100644
index 0000000000..553d5aa759
--- /dev/null
+++ b/libavutil/jni.c
@@ -0,0 +1,78 @@
+/*
+ * JNI public API functions
+ *
+ * Copyright (c) 2015-2016 Matthieu Bouron <matthieu.bouron stupeflix.com>
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#include "config.h"
+
+#include <stdlib.h>
+
+#include "libavutil/jni.h"
+#include "libavutil/error.h"
+
+#if CONFIG_JNI
+#include <jni.h>
+#include <pthread.h>
+
+#include "libavutil/log.h"
+
+static void *java_vm;
+static pthread_mutex_t lock = PTHREAD_MUTEX_INITIALIZER;
+
+int av_jni_set_jvm(void *vm, void *log_ctx)
+{
+    int ret = 0;
+
+    pthread_mutex_lock(&lock);
+    if (java_vm == NULL) {
+        java_vm = vm;
+    } else if (java_vm != vm) {
+        ret = AVERROR(EINVAL);
+        av_log(log_ctx, AV_LOG_ERROR, "A Java virtual machine has already been set");
+    }
+    pthread_mutex_unlock(&lock);
+
+    return ret;
+}
+
+void *av_jni_get_jvm(void *log_ctx)
+{
+    void *vm;
+
+    pthread_mutex_lock(&lock);
+    vm = java_vm;
+    pthread_mutex_unlock(&lock);
+
+    return vm;
+}
+
+#else
+
+int av_jni_set_jvm(void *vm, void *log_ctx)
+{
+    return AVERROR(ENOSYS);
+}
+
+void *av_jni_get_jvm(void *log_ctx)
+{
+    return NULL;
+}
+
+#endif
diff --git a/libavutil/jni.h b/libavutil/jni.h
new file mode 100644
index 0000000000..700960dbb8
--- /dev/null
+++ b/libavutil/jni.h
@@ -0,0 +1,46 @@
+/*
+ * JNI public API functions
+ *
+ * Copyright (c) 2015-2016 Matthieu Bouron <matthieu.bouron stupeflix.com>
+ *
+ * This file is part of FFmpeg.
+ *
+ * FFmpeg is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * FFmpeg is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with FFmpeg; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
+ */
+
+#ifndef AVUTIL_JNI_H
+#define AVUTIL_JNI_H
+
+/*
+ * Manually set a Java virtual machine which will be used to retrieve the JNI
+ * environment. Once a Java VM is set it cannot be changed afterwards, meaning
+ * you can call multiple times av_jni_set_java_vm with the same Java VM pointer
+ * however it will error out if you try to set a different Java VM.
+ *
+ * @param vm Java virtual machine
+ * @param log_ctx context used for logging, can be NULL
+ * @return 0 on success, < 0 otherwise
+ */
+int av_jni_set_jvm(void *vm, void *log_ctx);
+
+/*
+ * Get the Java virtual machine which has been set with av_jni_set_java_vm.
+ *
+ * @param vm Java virtual machine
+ * @return a pointer to the Java virtual machine
+ */
+void *av_jni_get_jvm(void *log_ctx);
+
+#endif /* AVUTIL_JNI_H */
diff --git a/libavcodec/ffjni.c b/libavutil/jniutils.c
similarity index 88%
rename from libavcodec/ffjni.c
rename to libavutil/jniutils.c
index e3cf24d3e2..89623af3f8 100644
--- a/libavcodec/ffjni.c
+++ b/libavutil/jniutils.c
@@ -31,7 +31,7 @@
 
 #include "config.h"
 #include "jni.h"
-#include "ffjni.h"
+#include "jniutils.h"
 
 static JavaVM *java_vm;
 static pthread_key_t current_env;
@@ -50,14 +50,14 @@ static void jni_create_pthread_key(void)
     pthread_key_create(&current_env, jni_detach_env);
 }
 
-JNIEnv *ff_jni_get_env(void *log_ctx)
+JNIEnv *avpriv_jni_get_env(void *log_ctx)
 {
     int ret = 0;
     JNIEnv *env = NULL;
 
     pthread_mutex_lock(&lock);
     if (java_vm == NULL) {
-        java_vm = av_jni_get_java_vm(log_ctx);
+        java_vm = av_jni_get_jvm(log_ctx);
     }
 
     if (!java_vm) {
@@ -96,7 +96,7 @@ done:
     return env;
 }
 
-char *ff_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx)
+char *avpriv_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx)
 {
     char *ret = NULL;
     const char *utf_chars = NULL;
@@ -126,7 +126,7 @@ char *ff_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx)
     return ret;
 }
 
-jstring ff_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *log_ctx)
+jstring avpriv_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *log_ctx)
 {
     jstring ret;
 
@@ -140,7 +140,7 @@ jstring ff_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *lo
     return ret;
 }
 
-int ff_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error, void *log_ctx)
+int avpriv_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error, void *log_ctx)
 {
     int ret = 0;
 
@@ -192,7 +192,7 @@ int ff_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error
     }
 
     if (string) {
-        name = ff_jni_jstring_to_utf_chars(env, string, log_ctx);
+        name = avpriv_jni_jstring_to_utf_chars(env, string, log_ctx);
         (*env)->DeleteLocalRef(env, string);
         string = NULL;
     }
@@ -214,7 +214,7 @@ int ff_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error
     }
 
     if (string) {
-        message = ff_jni_jstring_to_utf_chars(env, string, log_ctx);
+        message = avpriv_jni_jstring_to_utf_chars(env, string, log_ctx);
         (*env)->DeleteLocalRef(env, string);
         string = NULL;
     }
@@ -251,7 +251,7 @@ done:
     return ret;
 }
 
-int ff_jni_exception_check(JNIEnv *env, int log, void *log_ctx)
+int avpriv_jni_exception_check(JNIEnv *env, int log, void *log_ctx)
 {
     int ret;
 
@@ -271,7 +271,7 @@ int ff_jni_exception_check(JNIEnv *env, int log, void *log_ctx)
     exception = (*env)->ExceptionOccurred(env);
     (*(env))->ExceptionClear((env));
 
-    if ((ret = ff_jni_exception_get_summary(env, exception, &message, log_ctx)) < 0) {
+    if ((ret = avpriv_jni_exception_get_summary(env, exception, &message, log_ctx)) < 0) {
         (*env)->DeleteLocalRef(env, exception);
         return ret;
     }
@@ -284,7 +284,7 @@ int ff_jni_exception_check(JNIEnv *env, int log, void *log_ctx)
     return -1;
 }
 
-int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx)
+int avpriv_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx)
 {
     int i, ret = 0;
     jclass last_clazz = NULL;
@@ -299,7 +299,7 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
             last_clazz = NULL;
 
             clazz = (*env)->FindClass(env, jfields_mapping[i].name);
-            if ((ret = ff_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
+            if ((ret = avpriv_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
                 goto done;
             }
 
@@ -320,7 +320,7 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
             switch(type) {
             case FF_JNI_FIELD: {
                 jfieldID field_id = (*env)->GetFieldID(env, last_clazz, jfields_mapping[i].method, jfields_mapping[i].signature);
-                if ((ret = ff_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
+                if ((ret = avpriv_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
                     goto done;
                 }
 
@@ -329,7 +329,7 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
             }
             case FF_JNI_STATIC_FIELD: {
                 jfieldID field_id = (*env)->GetStaticFieldID(env, last_clazz, jfields_mapping[i].method, jfields_mapping[i].signature);
-                if ((ret = ff_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
+                if ((ret = avpriv_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
                     goto done;
                 }
 
@@ -338,7 +338,7 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
             }
             case FF_JNI_METHOD: {
                 jmethodID method_id = (*env)->GetMethodID(env, last_clazz, jfields_mapping[i].method, jfields_mapping[i].signature);
-                if ((ret = ff_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
+                if ((ret = avpriv_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
                     goto done;
                 }
 
@@ -347,7 +347,7 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
             }
             case FF_JNI_STATIC_METHOD: {
                 jmethodID method_id = (*env)->GetStaticMethodID(env, last_clazz, jfields_mapping[i].method, jfields_mapping[i].signature);
-                if ((ret = ff_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
+                if ((ret = avpriv_jni_exception_check(env, mandatory, log_ctx)) < 0 && mandatory) {
                     goto done;
                 }
 
@@ -367,13 +367,13 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
 done:
     if (ret < 0) {
         /* reset jfields in case of failure so it does not leak references */
-        ff_jni_reset_jfields(env, jfields, jfields_mapping, global, log_ctx);
+        avpriv_jni_reset_jfields(env, jfields, jfields_mapping, global, log_ctx);
     }
 
     return ret;
 }
 
-int ff_jni_reset_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx)
+int avpriv_jni_reset_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx)
 {
     int i;
 
diff --git a/libavcodec/ffjni.h b/libavutil/jniutils.h
similarity index 84%
rename from libavcodec/ffjni.h
rename to libavutil/jniutils.h
index 6027bac0ab..2be401c974 100644
--- a/libavcodec/ffjni.h
+++ b/libavutil/jniutils.h
@@ -20,8 +20,8 @@
  * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA
  */
 
-#ifndef AVCODEC_FFJNI_H
-#define AVCODEC_FFJNI_H
+#ifndef AVUTIL_JNIUTILS_H
+#define AVUTIL_JNIUTILS_H
 
 #include <jni.h>
 
@@ -37,7 +37,7 @@
  * @param log_ctx context used for logging, can be NULL
  * @return the JNI environment on success, NULL otherwise
  */
-JNIEnv *ff_jni_get_env(void *log_ctx);
+JNIEnv *avpriv_jni_get_env(void *log_ctx);
 
 /*
  * Convert a jstring to its utf characters equivalent.
@@ -48,7 +48,7 @@ JNIEnv *ff_jni_get_env(void *log_ctx);
  * @return a pointer to an array of unicode characters on success, NULL
  * otherwise
  */
-char *ff_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx);
+char *avpriv_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx);
 
 /*
  * Convert utf chars to its jstring equivalent.
@@ -58,7 +58,7 @@ char *ff_jni_jstring_to_utf_chars(JNIEnv *env, jstring string, void *log_ctx);
  * @param log_ctx context used for logging, can be NULL
  * @return a Java string object on success, NULL otherwise
  */
-jstring ff_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *log_ctx);
+jstring avpriv_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *log_ctx);
 
 /*
  * Extract the error summary from a jthrowable in the form of "className: errorMessage"
@@ -70,7 +70,7 @@ jstring ff_jni_utf_chars_to_jstring(JNIEnv *env, const char *utf_chars, void *lo
  * @param log_ctx context used for logging, can be NULL
  * @return 0 on success, < 0 otherwise
  */
-int ff_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error, void *log_ctx);
+int avpriv_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error, void *log_ctx);
 
 /*
  * Check if an exception has occurred,log it using av_log and clear it.
@@ -80,34 +80,30 @@ int ff_jni_exception_get_summary(JNIEnv *env, jthrowable exception, char **error
  * 0 disables logging, != 0 enables logging
  * @param log_ctx context used for logging, can be NULL
  */
-int ff_jni_exception_check(JNIEnv *env, int log, void *log_ctx);
+int avpriv_jni_exception_check(JNIEnv *env, int log, void *log_ctx);
 
 /*
  * Jni field type.
  */
 enum FFJniFieldType {
-
     FF_JNI_CLASS,
     FF_JNI_FIELD,
     FF_JNI_STATIC_FIELD,
     FF_JNI_METHOD,
     FF_JNI_STATIC_METHOD
-
 };
 
 /*
  * Jni field describing a class, a field or a method to be retrieved using
- * the ff_jni_init_jfields method.
+ * the avpriv_jni_init_jfields method.
  */
 struct FFJniField {
-
     const char *name;
     const char *method;
     const char *signature;
     enum FFJniFieldType type;
     int offset;
     int mandatory;
-
 };
 
 /*
@@ -124,7 +120,7 @@ struct FFJniField {
  * @param log_ctx context used for logging, can be NULL
  * @return 0 on success, < 0 otherwise
  */
-int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx);
+int avpriv_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx);
 
 /*
  * Delete class references, field ids and method ids of an arbitrary structure.
@@ -140,6 +136,6 @@ int ff_jni_init_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfi
  * @param log_ctx context used for logging, can be NULL
  * @return 0 on success, < 0 otherwise
  */
-int ff_jni_reset_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx);
+int avpriv_jni_reset_jfields(JNIEnv *env, void *jfields, const struct FFJniField *jfields_mapping, int global, void *log_ctx);
 
-#endif /* AVCODEC_FFJNI_H */
+#endif /* AVCODEC_JNIUTILS_H */
-- 
2.43.1



More information about the ffmpeg-devel mailing list