[Mplayer-cvslog] CVS: main/DOCS/tech vop.txt,1.21,1.22

[D Richard Felker III]

On Sun, Nov 03, 2002 at 10:17:45PM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Sunday 03 November 2002 20:47, D Richard Felker III wrote:
> > On Sun, Nov 03, 2002 at 11:05:51AM +0100, Michael Niedermayer CVS wrote:
> > > Update of /cvsroot/mplayer/main/DOCS/tech
> > > In directory mail:/var/tmp.root/cvs-serv14421/DOCS/tech
> > >
> > > Modified Files:
> > > 	vop.txt
> > > Log Message:
> > > pp docs
> [...]
> > > commandline option) +    see `mplayer -vop pp=help`
> >
> > Did you see my remark that this exits mplayer with the terminal in an
> > unusable state? 
> yes
> 
> > Has this been worked around?
> no, i was not sure how to (mplayer_exit() isnt in mencoder & mencoder_exit() 
> isnt in mplayer) and the whole *exit() in a filter is lame IMHO, so perhaps a 
> -pp-help option might be ok but i dunno if its a good idea to duplicate the 
> help text in the manpage & -pp-help
> ... but being able to just do a printf(pp_help); might be nice for some 
> projects which use libpostproc.so ...

I disagree. I don't think help text for the lib belongs in the lib
itself. Rather it belongs in docs where it can be read without running
the code, easily translated to different languages, etc. I'm not
saying to duplicate it in the man page, but rather remove it from the
code entirely.

Rich