diff mbox

[FFmpeg-devel] af_hdcd: add flags to AVOption defs

Message ID 1470066964-1607-2-git-send-email-pburt0@gmail.com
State Accepted
Commit e0b8cba0580e8a1ccf02f4c6c9bc1f448eec1ac8
Headers show

Commit Message

Burt P Aug. 1, 2016, 3:56 p.m. UTC 
Signed-off-by: Burt P <pburt0@gmail.com>
---
 libavfilter/af_hdcd.c | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

Comments

Timothy Gu Aug. 3, 2016, 5:43 a.m. UTC | #1 
On Mon, Aug 1, 2016 at 8:56 AM Burt P <pburt0@gmail.com> wrote:

> Signed-off-by: Burt P <pburt0@gmail.com>
> ---
>  libavfilter/af_hdcd.c | 5 +++--
>  1 file changed, 3 insertions(+), 2 deletions(-)
>

Applied. You might also want to look into using AV_OPT_TYPE_BOOL for these
two options.

Timothy
diff mbox

Patch

diff --git a/libavfilter/af_hdcd.c b/libavfilter/af_hdcd.c
index 2d7fe8d..7c90861 100644
--- a/libavfilter/af_hdcd.c
+++ b/libavfilter/af_hdcd.c
@@ -903,11 +903,12 @@  typedef struct HDCDContext {
 } HDCDContext;
 
 #define OFFSET(x) offsetof(HDCDContext, x)
+#define A AV_OPT_FLAG_AUDIO_PARAM|AV_OPT_FLAG_FILTERING_PARAM
 static const AVOption hdcd_options[] = {
     { "process_stereo", "Process stereo channels together. Only apply target_gain when both channels match.",
-        OFFSET(process_stereo), AV_OPT_TYPE_INT, { .i64 = HDCD_PROCESS_STEREO_DEFAULT }, 0, 1, 0 },
+        OFFSET(process_stereo), AV_OPT_TYPE_INT, { .i64 = HDCD_PROCESS_STEREO_DEFAULT }, 0, 1, A },
     { "force_pe", "Always extend peaks above -3dBFS even when PE is not signaled.",
-        OFFSET(force_pe), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, 0 },
+        OFFSET(force_pe), AV_OPT_TYPE_INT, { .i64 = 0 }, 0, 1, A },
     {NULL}
 };