[MPlayer-cvslog] r26723 - in trunk: Makefile av_opts.c av_opts.h

[Diego Biurrun]

On Fri, Aug 15, 2008 at 02:06:33PM +0200, Michael Niedermayer wrote:
> On Thu, Aug 14, 2008 at 06:01:27PM +0200, Diego Biurrun wrote:
> > On Sat, May 10, 2008 at 08:54:11PM +0200, michael wrote:
> > > 
> > > Log:
> > > AVOptions support.
> > 
> > Documentation is missing and none has been written in the 3 months since
> > it was committed.  This is a policy violation.  I will back this out if
> > nobody steps up to fill the gaps by the end of next week.
> > 
> > Before anybody asks: I am not volunteering to do it myself.  I have no
> > idea what it does nor the motivation to RTFS in order to find out.
> 
> Ive documented the lavc encoder part of it now, the other parts behave
> identically.

Nice, but that is still only 1/4, the rest needs to be documented as
well.  I will extend the deadline to September as a sign of good will,
but if it is not complete by then, I will back this out.

> And it would be a pitty if it where reverted considering that AVOptions
> allow access to all parameters without them being explicitly listed
> in vd/ve_ffmpeg/lavc.c. And considering that there is noone who does
> or would maintain these redundant lists.

It's always sad to see code go, but these are the rules.

Diego