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

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

[Matthieu Bouron]

---
 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

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

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

[Matthieu Bouron]

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.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".

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

[Matthieu Bouron]

On Mon, Apr 15, 2024 at 07:02:36PM +0200, Matthieu Bouron wrote:
> 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.

Patch applied.

-- 
Matthieu B.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".