Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

Quoting Vittorio Giovara (2016-06-16 00:43:26) > On Wed, Jun 15, 2016 at 5:31 PM, Hendrik Leppkes wrote: > > On Wed, Jun 15, 2016 at 11:06 PM, Vittorio Giovara > > wrote: > >> On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

Quoting Luca Barbato (2016-06-16 01:34:39) > On 16/06/16 00:43, Vittorio Giovara wrote: > > Finally with the addition of 4/5 we will be able to use these names > > when calling the conversion tool too, so we will finally be compatible > > with ourselves. > > As said, this is a bugfix, at least

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

Quoting Luca Barbato (2016-06-16 19:09:45) > On 16/06/16 18:56, Vittorio Giovara wrote: > > The symbols with the unfortunate name were added in 4a66422 (17 Sep > > 2015), they were not part of the 11 release, nor is the API in > > question ever released in any libav release. > > Then we can

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On 20/06/16 20:18, Vittorio Giovara wrote: > On Thu, Jun 16, 2016 at 1:27 PM, Vittorio Giovara > wrote: >> On Thu, Jun 16, 2016 at 1:09 PM, Luca Barbato wrote: >>> On 16/06/16 18:56, Vittorio Giovara wrote: The symbols with the unfortunate

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 1:27 PM, Vittorio Giovara wrote: > On Thu, Jun 16, 2016 at 1:09 PM, Luca Barbato wrote: >> On 16/06/16 18:56, Vittorio Giovara wrote: >>> The symbols with the unfortunate name were added in 4a66422 (17 Sep >>> 2015), they

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 1:09 PM, Luca Barbato wrote: > On 16/06/16 18:56, Vittorio Giovara wrote: >> The symbols with the unfortunate name were added in 4a66422 (17 Sep >> 2015), they were not part of the 11 release, nor is the API in >> question ever released in any libav

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On 16/06/16 18:56, Vittorio Giovara wrote: > The symbols with the unfortunate name were added in 4a66422 (17 Sep > 2015), they were not part of the 11 release, nor is the API in > question ever released in any libav release. Then we can change it as you like IMO. lu

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Thu, Jun 16, 2016 at 4:31 AM, Diego Biurrun wrote: > On Wed, Jun 15, 2016 at 03:08:57PM -0400, Vittorio Giovara wrote: >> On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes wrote: >> > On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 03:08:57PM -0400, Vittorio Giovara wrote: > On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes wrote: > > On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato wrote: > >> On 15/06/16 20:25, Vittorio Giovara wrote: > >>> This allows to

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On 16/06/16 00:43, Vittorio Giovara wrote: > Finally with the addition of 4/5 we will be able to use these names > when calling the conversion tool too, so we will finally be compatible > with ourselves. As said, this is a bugfix, at least one of the names before this patch looks like a typo. lu

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 5:31 PM, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 11:06 PM, Vittorio Giovara > wrote: >> On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes wrote: >>> On Wed, Jun 15, 2016 at 9:08 PM, Vittorio

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 11:06 PM, Vittorio Giovara wrote: > On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes wrote: >> On Wed, Jun 15, 2016 at 9:08 PM, Vittorio Giovara >> wrote: >>> On Wed, Jun 15, 2016 at 3:00 PM,

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 3:49 PM, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 9:08 PM, Vittorio Giovara > wrote: >> On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes wrote: >>> On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 9:08 PM, Vittorio Giovara wrote: > On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes wrote: >> On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato wrote: >>> On 15/06/16 20:25, Vittorio Giovara wrote:

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 3:00 PM, Hendrik Leppkes wrote: > On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato wrote: >> On 15/06/16 20:25, Vittorio Giovara wrote: >>> This allows to directly use av_color_*_name() return value as >>> command line parameters for

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On Wed, Jun 15, 2016 at 8:53 PM, Luca Barbato wrote: > On 15/06/16 20:25, Vittorio Giovara wrote: >> This allows to directly use av_color_*_name() return value as >> command line parameters for the encoder. >> --- >> libavutil/pixdesc.c | 8 >> libavutil/version.h |

Re: [libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

On 15/06/16 20:25, Vittorio Giovara wrote: > This allows to directly use av_color_*_name() return value as > command line parameters for the encoder. > --- > libavutil/pixdesc.c | 8 > libavutil/version.h | 2 +- > 2 files changed, 5 insertions(+), 5 deletions(-) I consider the previous

[libav-devel] [PATCH 2/5] pixdesc: Make sure color properties names match x264

This allows to directly use av_color_*_name() return value as command line parameters for the encoder. --- libavutil/pixdesc.c | 8 libavutil/version.h | 2 +- 2 files changed, 5 insertions(+), 5 deletions(-) diff --git a/libavutil/pixdesc.c b/libavutil/pixdesc.c index 209d107..216452c