[FFmpeg-devel] [PATCH] avcodec: Add AVClass to AVCodecParameters

[wm4]

On Tue, 10 May 2016 08:24:35 +0200
Hendrik Leppkes <h.leppkes at gmail.com> wrote:

> On Tue, May 10, 2016 at 2:07 AM, Michael Niedermayer
> <michael at niedermayer.cc> wrote:
> > Allow enumeration, read and write of fields without requiring #if on versions
> > All other public structures can be accessed through AVOptions
> >
> > TODO: add all fields to AVOption table
> > TODO: bump version
> > TODO: update APIChanges  
> 
> 
> I don't like this, the struct is pretty cleanly defined and unlikely
> to be extended much over time.
> Most other structs have AVOptions so the CLI can interact with it, but
> this struct is not meant to be modified by users, its just a direct
> line of communication between demuxer->decoder or encoder->muxer.

+1

It's completely inappropriate and you'd end up with every struct having
AVClass. Stop trying to pretend ffmpeg is written in python or
something.