[FFmpeg-devel] [PATCH 1/2] lavu: Add JEDEC P22 color primaries

[Rostislav Pehlivanov]

On 1 December 2016 at 17:25, Vittorio Giovara <vittorio.giovara at gmail.com>
wrote:

> On Wed, Nov 30, 2016 at 5:48 PM, Ronald S. Bultje <rsbultje at gmail.com>
> wrote:
> > Hi,
> >
> > On Wed, Nov 30, 2016 at 5:26 PM, Andreas Cadhalpun
> > <andreas.cadhalpun at googlemail.com> wrote:
> >>
> >> On 30.11.2016 22:55, Ronald S. Bultje wrote:
> >> > On Wed, Nov 30, 2016 at 4:51 PM, Andreas Cadhalpun <
> >> > andreas.cadhalpun at googlemail.com> wrote:
> >> >
> >> >> On 30.11.2016 19:16, Vittorio Giovara wrote:
> >> >> You can't just add a gap like that.
> >> >> The current code assumes that the numbers are consecutive, like e.g.
> >> >> the
> >> >> naming of AVCOL_PRI_NB suggests.
> >> >
> >> >
> >> > No, we've had gaps in these before.
> >>
> >> In AVColorPrimaries?
> >
> >
> > I seemed to remember, looked it up and appeared to be wrong, so scrap
> that
> > :)
>
> Ok, so I can take care of the "holes" this patch would bring for lavc,
> would anybody help me with the remaining occurrences (namely in lavf
> and lavfi)?
> --


Why do you insist on having a hole in the first place?