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

Vittorio Giovara vittorio.giovara at gmail.com
Thu Dec 1 19:25:47 EET 2016


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)?
-- 
Vittorio


More information about the ffmpeg-devel mailing list