diff mbox series

[FFmpeg-devel,09/20] fftools/cmdutils: check valid flags for OPT_TYPE_FUNC

Message ID 20231218095722.25879-9-anton@khirnov.net
State Accepted
Commit 579238226901285eff24ef88ae1a53b7f166adf5
Headers show
Series [FFmpeg-devel,01/20] fftools/ffmpeg_filter: only set framerate for video | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Anton Khirnov Dec. 18, 2023, 9:57 a.m. UTC 
SPEC and OFFSET do not make sense for functions.
---
 fftools/cmdutils.c | 3 +++
 1 file changed, 3 insertions(+)
diff mbox series

Patch

diff --git a/fftools/cmdutils.c b/fftools/cmdutils.c
index 6ca2efef4a..a52c7c5ae4 100644
--- a/fftools/cmdutils.c
+++ b/fftools/cmdutils.c
@@ -482,6 +482,9 @@  static void check_options(const OptionDef *po)
         if (po->flags & OPT_PERFILE)
             av_assert0(po->flags & (OPT_INPUT | OPT_OUTPUT));
 
+        if (po->type == OPT_TYPE_FUNC)
+            av_assert0(!(po->flags & (OPT_FLAG_OFFSET | OPT_FLAG_SPEC)));
+
         // OPT_FUNC_ARG can only be ser for OPT_TYPE_FUNC
         av_assert0((po->type == OPT_TYPE_FUNC) || !(po->flags & OPT_FUNC_ARG));