[FFmpeg-devel] [PATCH v3] lavu: add an AV_FRAME_DATA_GAMMA side data type
atomnuker at gmail.com
Wed Nov 8 22:59:39 EET 2017
On 8 November 2017 at 20:52, Paul B Mahol <onemda at gmail.com> wrote:
> On 11/8/17, Nicolas George <george at nsup.org> wrote:
> > L'octidi 18 brumaire, an CCXXVI, Hendrik Leppkes a ecrit :
> >> 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%
> > overhead.
> > 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.
> Maybe I promised a fork, but looks like people are already forking FFmpeg
> because of people like you, latest one being ffmpeg-mpv.
> There should be new call for voting commitee to drop you, Michael and Carl
> from veto voting commitee. You guys should not block others work.
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
Chill, different opinions important.
I don't particularly care about the patch really, its only exposed by png
and never used outside of joke images.
More information about the ffmpeg-devel