[FFmpeg-devel] [PATCH v3] lavu: add an AV_FRAME_DATA_GAMMA side data type
george at nsup.org
Wed Nov 8 22:36:27 EET 2017
L'octidi 18 brumaire, an CCXXVI, Hendrik Leppkes a écrit :
> In the meantime, it does no real harm to add one more sidedata type to
> the numerous list of types we already have, and lets people carry on
> with their work.
It does harm, look at the other patch: 5 lines of useless clutter out of
10 lines for the feature, using side data for that features has a 100%
And that is not only for the code here, it has the same consequences for
the applications that will use that API as well: they have to do the
same errors checks, they have to handle a type-pruned pointer, check the
size of the data, etc. This is a terrible API, there is no doubt that
people would prefer a single field.
By all means let us discuss all this. But if Paul maintains his veto, I
maintain mine. Please persuade Paul to behave like an adult of make good
on his trice-repeated promise of forking.
-------------- next part --------------
A non-text attachment was scrubbed...
Size: 833 bytes
Desc: Digital signature
More information about the ffmpeg-devel