[FFmpeg-devel] [PATCH v3] lavu: add an AV_FRAME_DATA_GAMMA side data type

Hendrik Leppkes h.leppkes at gmail.com
Wed Nov 8 22:44:59 EET 2017


On Wed, Nov 8, 2017 at 9:36 PM, Nicolas George <george at nsup.org> wrote:
> This is a terrible API, there is no doubt that
> people would prefer a single field.
>

I don't. We've done a lot of work to remove fields from generic data
structures which are used by single codecs on semi-obscure cases. The
fact alone that after years of avcodec and a PNG decoder this field
wasn't asked for regularly alone speaks to me that its not generic
enough to warrant inclusion in AVFrame.

Certainly it would make for nicer code if you could access every field
directly, but if we do that for every piece of data provided by all
random single decoders, we have a nightmare of a frame structure, with
hundreds of fields that only ever are applicable in a minority of
cases.

- Hendrik


More information about the ffmpeg-devel mailing list