[MPlayer-cvslog] r26727 - trunk/libmpdemux/muxer_lavf.c

[Michael Niedermayer]

On Sat, May 10, 2008 at 10:25:17PM -0500, compn wrote:
> On Sun, 11 May 2008 03:23:11 +0200 Michael Niedermayer <michaelni at gmx.at> wrote:
> 
> >On Sat, May 10, 2008 at 09:10:12PM -0500, compn wrote:
> >> On Sat, 10 May 2008 21:22:41 +0200 (CEST) michael <subversion at mplayerhq.hu> wrote:
> >> 
> >> >Author: michael
> >> >Date: Sat May 10 21:22:41 2008
> >> >New Revision: 26727
> >> >
> >> >Log:
> >> >AVOptions support for lavf muxing.
> >> 
> >> is this the start of 'removing all duplicated AVoptions from mplayer/mencoder' ? hehe
> >
> >Eventually maybe ...
> >
> >At the moment the most annoying thing is the syntax
> >
> >-lavfopts AAA=BBB:CCC=DDD
> >vs.
> >-lavfopts o=AAA=BBB,CCC=DDD
> >
> >This should be fixed first so that unkown options are fed into the AVOptions
> >code instead of requireing the new and ugly syntax.
> >
> 
> btw what happens when 
> mencoder -ovc lavc -lavcopts vcodec=mpeg4:o=vcodec=mpeg2video ?

You cant set the codec via AVOption currently IIRC. Also some options
contain special cases in mplayer these as well will not work properly
with AVOptions ATM. That special cases have to be changed one by one
to be compatible with AVOptions where its possible to do so.

Next steps are (and i do not volunteer ...)
* do something about the ugly o= syntax, someone knowing all the option
  handling code likely should be able to fix that in less time than i
  could ...
* go over the options one by one and test if they work with AVOptions
  and remove the old ones if they do
* fix whatever does not work with AVOptions but can be easily and
  cleanly be made to work.


> 
> also need docs update...

Ill write and commit that right after uoti splits one of his messup commits
according to the policy.
Until then i act by my own common sense like uoti does.
And my common sense says developers code they dont write docs :)

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Those who are too smart to engage in politics are punished by being
governed by those who are dumber. -- Plato 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-cvslog/attachments/20080511/3dcadcc2/attachment.pgp>