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

Derek Buitenhuis derek.buitenhuis at gmail.com
Tue May 10 12:17:12 CEST 2016


On 5/10/2016 7:24 AM, Hendrik Leppkes wrote:
> 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.

To me this mostly still feels like trying to make a demuxing library
something it isn't. It demuxes and muxes.

You can argue that it should be easier than decoding a frame to get this
info (which I don't personally think is much trouble at all). However,
jamming it into libavformat for reasons like "we've always done this" and
"there's no better place" is not good design IMO.

It sounds like what you really want is a higher level "easy" API. Don't
jam it in to existing decoupled APIs because it is convenient, IMO. That's
how horrible things like having ffmpeg.c functionality in libavformat came
to exist, and a decade of misery followed.

Or, of course, merge the two libraries and drop pretenses of having a
"demuxing" API at all.

- Derek


More information about the ffmpeg-devel mailing list