diff mbox

[FFmpeg-devel,06/12] ffmpeg_opt: Make get_timecode actually return errors

Message ID 20170706182842.7785-7-derek.buitenhuis@gmail.com
State Accepted
Commit d74ba68acf30ec5395cb0c82007cc10c02d2e028
Headers show

Commit Message

Derek Buitenhuis July 6, 2017, 6:28 p.m. UTC 
Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
---
 ffmpeg_opt.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Michael Niedermayer July 7, 2017, 12:15 a.m. UTC | #1 
On Thu, Jul 06, 2017 at 07:28:36PM +0100, Derek Buitenhuis wrote:
> Signed-off-by: Derek Buitenhuis <derek.buitenhuis@gmail.com>
> ---
>  ffmpeg_opt.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)

probably ok

[...]
diff mbox

Patch

diff --git a/ffmpeg_opt.c b/ffmpeg_opt.c
index 9b7e8c74b9..989391bb34 100644
--- a/ffmpeg_opt.c
+++ b/ffmpeg_opt.c
@@ -3072,7 +3072,7 @@  static int opt_timecode(void *optctx, const char *opt, const char *arg)
     if (ret >= 0)
         ret = av_dict_set(&o->g->codec_opts, "gop_timecode", arg, 0);
     av_free(tcr);
-    return 0;
+    return ret;
 }
 
 static int opt_channel_layout(void *optctx, const char *opt, const char *arg)