[FFmpeg-devel] [PATCH] avcodec/jni: fix mixed declaration and code

Matthieu Bouron matthieu.bouron at gmail.com
Mon Apr 15 20:02:36 EEST 2024


On Thu, Apr 04, 2024 at 10:07:25PM +0200, Matthieu Bouron wrote:
> ---
>  libavcodec/jni.c | 4 +++-
>  1 file changed, 3 insertions(+), 1 deletion(-)
> 
> diff --git a/libavcodec/jni.c b/libavcodec/jni.c
> index 1193c608c3..fcb4837413 100644
> --- a/libavcodec/jni.c
> +++ b/libavcodec/jni.c
> @@ -84,11 +84,13 @@ void *av_jni_get_java_vm(void *log_ctx)
>  int av_jni_set_android_app_ctx(void *app_ctx, void *log_ctx)
>  {
>  #if CONFIG_JNI
> +    jobjectRefType type;
> +
>      JNIEnv *env = ff_jni_get_env(log_ctx);
>      if (!env)
>          return AVERROR(EINVAL);
>  
> -    jobjectRefType type = (*env)->GetObjectRefType(env, app_ctx);
> +    type = (*env)->GetObjectRefType(env, app_ctx);
>      if (type != JNIGlobalRefType) {
>          av_log(log_ctx, AV_LOG_ERROR, "Application context must be passed as a global reference");
>          return AVERROR(EINVAL);
> -- 
> 2.44.0
> 

I will apply the patch in a few days if there is no objection.


-- 
Matthieu B.


More information about the ffmpeg-devel mailing list