[PR] avutil/opt: reformat set_string_binary (PR #24161)

Leo Izen via ffmpeg-devel <[email protected]>
Newsgroups gmane.comp.video.ffmpeg.devel
Message-ID <178680424439.59.2362448371495714183@29965ddac10e>
PR #24161 opened by Leo Izen (Traneptora)
URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24161
Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/24161.patch

Fix the code formatting in libavutil/opt.c's static function
set_string_binary after bd68a6130495fe17bf33e0a9ddc053baae18c81f.

No functional changes.

Signed-off-by: Leo Izen <[email protected]>


>From 679ce63d4723f9d630c494f166e170dbd193de93 Mon Sep 17 00:00:00 2001
From: Leo Izen <[email protected]>
Date: Sat, 15 Aug 2026 10:28:04 -0400
Subject: [PATCH] avutil/opt: reformat set_string_binary

Fix the code formatting in libavutil/opt.c's static function
set_string_binary after bd68a6130495fe17bf33e0a9ddc053baae18c81f.

No functional changes.

Signed-off-by: Leo Izen <[email protected]>
---
 libavutil/opt.c | 12 +++++++-----
 1 file changed, 7 insertions(+), 5 deletions(-)

diff --git a/libavutil/opt.c b/libavutil/opt.c
index a4709740bd..6fa1ec9929 100644
--- a/libavutil/opt.c
+++ b/libavutil/opt.c
@@ -369,8 +369,8 @@ static int set_string_binary(void *obj, const AVOption *o, const char *val, uint
 
     if (dst) {
         lendst = (int *)(dst + 1);
-    av_freep(dst);
-    *lendst = 0;
+        av_freep(dst);
+        *lendst = 0;
     }
 
     if (!val || !(len = strlen(val)))
@@ -392,11 +392,13 @@ static int set_string_binary(void *obj, const AVOption *o, const char *val, uint
         }
         *ptr++ = (a << 4) | b;
     }
+
     if (dst) {
-    *dst    = bin;
-    *lendst = len;
-    } else
+        *dst    = bin;
+        *lendst = len;
+    } else {
         av_free(bin);
+    }
 
     return 0;
 }
-- 
2.52.0

_______________________________________________
ffmpeg-devel mailing list -- [email protected]
To unsubscribe send an email to [email protected]
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.