[MPlayer-dev-eng] [PATCH] Support ffmpeg per-codec avoptions

[Philip Langdale]

On Sun, 24 Apr 2011 08:18:43 +0200
Reimar Döffinger <Reimar.Doeffinger at gmx.de> wrote:

> > 
> > It would be a little awkward. You'd have to pass the lavc_codec in
> > as a separate argument (can't set the codec in on your own -
> > avcodec_open has to do it) and then give parse_avopts knowledge of
> > the types - right now it takes a void*.
> 
> I somehow doubt that. At least the priv_data check should be easy to
> avoid by making parse_avopts return an error if the first argument is
> NULL...

I see what you mean. I thought you wanted it to move all the logic into
parse_avopts so that you would only call it once.

I'll update the diff.

--phil