[MPlayer-dev-eng] [PATCH] verbosity reduction around CPU flags

[Dominik 'Rathann' Mierzejewski]

On Monday, 20 November 2006 at 00:40, Diego Biurrun wrote:
> On Sun, Nov 12, 2006 at 03:10:34PM +0100, Diego Biurrun wrote:
> > On Fri, Nov 10, 2006 at 10:00:47PM +0100, Roberto Togni wrote:
> > > On Fri, 10 Nov 2006 20:19:36 +0100
> > > Diego Biurrun <diego at biurrun.de> wrote:
> > > 
> > > > Here's a patch that makes MPlayer print info about the CPU flags it
> > > > encountered in verbose mode only.  Note that the info is triplicated.
> > > > This should be refactored someday ...
> > > 
> > > As already said in answer to
> > > Subject: [MPlayer-dev-eng][PATCH] verbosity reduction in streaming,
> > > input, CPU detection
> > > Date: Sun, 24 Sep 2006 18:22:39 +0200
> > > I don't see any good reason to remove them.
> > > They are informative and non intrusive (printed at the beginning, so
> > > nothing is scrolled out because of them).
> > 
> > They are useful only for debugging purposes.  IMO debug output should be
> > reserved for verbose mode and the default output should just contain
> > information about the movie itself (codec, fps, bpp, samplerate,
> > available tracks) and some basic information from the output drivers.
> 
> Do we opinions from other people on this matter?

CPU flags need to be present with -v. I don't have a strong opinion at
default verbosity either way.

Regards,
R.

-- 
MPlayer developer and RPMs maintainer: http://rpm.greysector.net/mplayer/
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
	-- from "Collected Sayings of Muad'Dib" by the Princess Irulan