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

2016-09-13 Thread Ronald S. Bultje 
Hi,

On Mon, Sep 12, 2016 at 1:46 PM, Ronald S. Bultje 
wrote:

> Hi,
>
> On Mon, Sep 12, 2016 at 1:44 PM, Vittorio Giovara <
> vittorio.giov...@gmail.com> wrote:
>
>> Allows to use values returned from API and from ffprobe directly.
>>
>> Signed-off-by: Vittorio Giovara 
>> ---
>>  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
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-09-12 Thread Ronald S. Bultje 
Hi,

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

> Allows to use values returned from API and from ffprobe directly.
>
> Signed-off-by: Vittorio Giovara 
> ---
>  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
___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

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

2016-09-12 Thread Vittorio Giovara 
Allows to use values returned from API and from ffprobe directly.

Signed-off-by: Vittorio Giovara 
---
 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

___
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel