[MPlayer-dev-eng] [PATCH] nostatus-line option

Ivan Kalvachev ikalvachev at gmail.com
Fri May 18 18:29:35 CEST 2007


2007/5/18, Diego Biurrun <diego at biurrun.de>:
> On Fri, May 18, 2007 at 06:13:59PM +0300, Ivan Kalvachev wrote:
> > 2007/5/18, Diego Biurrun <diego at biurrun.de>:
> > > Attila wanted it, I've been meaning to implement it for ages, here it
> > > is: A small option that turns off displaying of the status line.  Useful
> > > if you wish to have full debug output but your terminal is slow or you
> > > wish to redirect output to a file without the status line crap.
> > >
> > > Comments welcome.  If somebody has a better name, that is also welcome.
> >
> > My futile attempt to explain what Attila really wanted ends up in
> > another bloatware option. You should've talked to him first.
>
> I did.  Are you going to tell me next that your English is better than
> mine?

You just don't get it.

Attila wanted logging _copy_ of the output to a file. But in a way so
the log file does not contain the thousands status lines that indicate
normal work. He does not  wanted to change the normal screen output in
any way.

Your patch can not do that.

> > How is your new option different that "-msglevel avsync=4" (afaik
> > status line is 5)
>
> It turns off the status line and nothing but the status line while
> leaving all the other avsync-related output untouched.

The rest of avsync related output is 4 messages: 2 DBG2 messages and 2
other messages that would be still visible with the above command as
they have lower levels.

Attila patch was about creating new status message type (MSGT) only
for the status line so the other avsync messages would still be
visible.
However it doesn't worth the effort, as if you want to see the avsync
verbose debug info you'd definitely want to see the avsync info from
the status line. After all that's the whole idea of having levels...

If you still want clean way to suppress only the status lines, then
I'd recommend you the Attila approach, as it just extends existing
infrastructure, allows more flexibility and doesn't add new random
global variables and options.



More information about the MPlayer-dev-eng mailing list