diff mbox

[FFmpeg-devel] vf_colorspace: Add BT-names for gamma22/28 transfer option

Message ID 20160912174402.90794-1-vittorio.giovara@gmail.com
State Accepted
Commit d41bfa9c0b109e6ef8310392cf8a5ee6741a5ea5
Headers show

Commit Message

Vittorio Giovara Sept. 12, 2016, 5:44 p.m. UTC
Allows to use values returned from API and from ffprobe directly.

Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
---
 libavfilter/vf_colorspace.c | 2 ++
 1 file changed, 2 insertions(+)

Comments

Ronald S. Bultje Sept. 12, 2016, 5:46 p.m. UTC | #1
Hi,

On Mon, Sep 12, 2016 at 1:44 PM, Vittorio Giovara <
vittorio.giovara@gmail.com> wrote:

> Allows to use values returned from API and from ffprobe directly.
>
> Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
> ---
>  libavfilter/vf_colorspace.c | 2 ++
>  1 file changed, 2 insertions(+)
>
> diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
> index 45fd917..b9ecb5f 100644
> --- a/libavfilter/vf_colorspace.c
> +++ b/libavfilter/vf_colorspace.c
> @@ -1055,7 +1055,9 @@ static const AVOption colorspace_options[] = {
>        OFFSET(user_trc),   AV_OPT_TYPE_INT, { .i64 = AVCOL_TRC_UNSPECIFIED
> },
>        AVCOL_TRC_RESERVED0, AVCOL_TRC_NB - 1, FLAGS, "trc" },
>      ENUM("bt709",        AVCOL_TRC_BT709,        "trc"),
> +    ENUM("bt470m",       AVCOL_TRC_GAMMA22,      "trc"),
>      ENUM("gamma22",      AVCOL_TRC_GAMMA22,      "trc"),
> +    ENUM("bt470bg",      AVCOL_TRC_GAMMA28,      "trc"),
>      ENUM("gamma28",      AVCOL_TRC_GAMMA28,      "trc"),
>      ENUM("smpte170m",    AVCOL_TRC_SMPTE170M,    "trc"),
>      ENUM("smpte240m",    AVCOL_TRC_SMPTE240M,    "trc"),
> --
> 2.9.3


OK.

(Can you push yourself or do you need one of us to do it for you?)

Ronald
Ronald S. Bultje Sept. 13, 2016, 2:26 p.m. UTC | #2
Hi,

On Mon, Sep 12, 2016 at 1:46 PM, Ronald S. Bultje <rsbultje@gmail.com>
wrote:

> Hi,
>
> On Mon, Sep 12, 2016 at 1:44 PM, Vittorio Giovara <
> vittorio.giovara@gmail.com> wrote:
>
>> Allows to use values returned from API and from ffprobe directly.
>>
>> Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
>> ---
>>  libavfilter/vf_colorspace.c | 2 ++
>>  1 file changed, 2 insertions(+)
>>
>> diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
>> index 45fd917..b9ecb5f 100644
>> --- a/libavfilter/vf_colorspace.c
>> +++ b/libavfilter/vf_colorspace.c
>> @@ -1055,7 +1055,9 @@ static const AVOption colorspace_options[] = {
>>        OFFSET(user_trc),   AV_OPT_TYPE_INT, { .i64 =
>> AVCOL_TRC_UNSPECIFIED },
>>        AVCOL_TRC_RESERVED0, AVCOL_TRC_NB - 1, FLAGS, "trc" },
>>      ENUM("bt709",        AVCOL_TRC_BT709,        "trc"),
>> +    ENUM("bt470m",       AVCOL_TRC_GAMMA22,      "trc"),
>>      ENUM("gamma22",      AVCOL_TRC_GAMMA22,      "trc"),
>> +    ENUM("bt470bg",      AVCOL_TRC_GAMMA28,      "trc"),
>>      ENUM("gamma28",      AVCOL_TRC_GAMMA28,      "trc"),
>>      ENUM("smpte170m",    AVCOL_TRC_SMPTE170M,    "trc"),
>>      ENUM("smpte240m",    AVCOL_TRC_SMPTE240M,    "trc"),
>> --
>> 2.9.3
>
>
> OK.
>

Pushed.

Ronald
diff mbox

Patch

diff --git a/libavfilter/vf_colorspace.c b/libavfilter/vf_colorspace.c
index 45fd917..b9ecb5f 100644
--- a/libavfilter/vf_colorspace.c
+++ b/libavfilter/vf_colorspace.c
@@ -1055,7 +1055,9 @@  static const AVOption colorspace_options[] = {
       OFFSET(user_trc),   AV_OPT_TYPE_INT, { .i64 = AVCOL_TRC_UNSPECIFIED },
       AVCOL_TRC_RESERVED0, AVCOL_TRC_NB - 1, FLAGS, "trc" },
     ENUM("bt709",        AVCOL_TRC_BT709,        "trc"),
+    ENUM("bt470m",       AVCOL_TRC_GAMMA22,      "trc"),
     ENUM("gamma22",      AVCOL_TRC_GAMMA22,      "trc"),
+    ENUM("bt470bg",      AVCOL_TRC_GAMMA28,      "trc"),
     ENUM("gamma28",      AVCOL_TRC_GAMMA28,      "trc"),
     ENUM("smpte170m",    AVCOL_TRC_SMPTE170M,    "trc"),
     ENUM("smpte240m",    AVCOL_TRC_SMPTE240M,    "trc"),