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

[mail at kraymer.de]

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?

I'm fine with it. Although I'm very used to the output as it is, I'm
pretty  sure I could live well without it. It's not printed but I still
know my  MPlayer build is fairly optimized ;)

Sebastian


PS: Maybe an empty line between MP Version and usage line would be nice,
not sure..