diff mbox series

[FFmpeg-devel,5/7] avutil/jni: remove unnecessary NULL checks before calling DeleteLocalRef()

Message ID 20240213230418.457056-6-matthieu.bouron@gmail.com
State New
Headers show
Series [FFmpeg-devel,1/7] avcodec: move ffjni to avutil/jniutils | expand

Checks

Context Check Description
andriy/configure_x86 warning Failed to apply patch
yinshiyou/configure_loongarch64 warning Failed to apply patch

Commit Message

Matthieu Bouron Feb. 13, 2024, 10:50 p.m. UTC 
Delete{Global,Local}Ref() already handle NULL.
---
 libavutil/jniutils.c | 14 +++-----------
 1 file changed, 3 insertions(+), 11 deletions(-)
diff mbox series

Patch

diff --git a/libavutil/jniutils.c b/libavutil/jniutils.c
index 89623af3f8..beedf24c0d 100644
--- a/libavutil/jniutils.c
+++ b/libavutil/jniutils.c
@@ -236,17 +236,9 @@  done:
     av_free(name);
     av_free(message);
 
-    if (class_class) {
-        (*env)->DeleteLocalRef(env, class_class);
-    }
-
-    if (exception_class) {
-        (*env)->DeleteLocalRef(env, exception_class);
-    }
-
-    if (string) {
-        (*env)->DeleteLocalRef(env, string);
-    }
+    (*env)->DeleteLocalRef(env, class_class);
+    (*env)->DeleteLocalRef(env, exception_class);
+    (*env)->DeleteLocalRef(env, string);
 
     return ret;
 }