[MPlayer-dev-eng] [PATCH] CPU info verbosity reduction

[Andrew Savchenko]

Hi,

On Saturday 21 February 2009 00:56, Ivan Kalvachev wrote:
> On 2/9/09, Diego Biurrun <diego at biurrun.de> wrote:
> > On Fri, Jan 30, 2009 at 10:50:10PM +0100, Diego Biurrun wrote:
> >> Here is a patch to make MPlayer print less information by
> >> default about the CPU it detects.  I consider this
> >> information suitable for verbose mode only.
> >
> > Applied.
>
> Please, revert.
>
> You can't bring back the strings using command line -v.
> Command line parsing is done after cpu detection.

Seconded. While this information is not vital, it is very nice for 
the user to see how CPU is identified and what instruction sets 
are in use. This string have rather strong psychological effect, 
users will say somewhat similar to: "Hey, my CPU is identified 
properly and all it's abilities are accounted for. That's cool!"

Yes, from performance point of view that string changes nothing, 
but as long as it demonstraits optimization level it should be 
present by default, imho.

-- 
Best regards,
Andrew
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20090221/2336f212/attachment.pgp>