[MPlayer-dev-eng] [PATCH] MSGT_IDENTIFY (was: option to skip slow -identify output)

Reynaldo H. Verdejo Pinochet reynaldo at opendot.cl
Mon Apr 24 09:22:44 CEST 2006


On Mon, Apr 24, 2006 at 01:43:50AM -0400, Reynaldo H. Verdejo Pinochet wrote:
> On Mon, Apr 24, 2006 at 07:54:12AM +0300, Oded Shimon wrote:
> > On Sat, Apr 22, 2006 at 09:46:03AM -0400, kiriuja wrote:
> > > On Saturday 22 April 2006 09:16 Oded Shimon wrote:
> > > > On Thu, Apr 20, 2006 at 06:13:21PM -0400, kiriuja wrote:
> > > > > +    mp_msg_levels[MSGT_IDENTIFY] = -1; // no -identify output by default
> > > > 
> > > > I dislike this hack.
> > > > 
> > > > IMO you should make all identify MSGL_V, and the very bad ones, MSGL_DBG1 
> > > > (or 2 or whatever it was).
> > > 
> > > Currently there is no identify output unless you give -identify, regardless
> > > of the global msglevel. There is no reason to change that. So -1 is the
> > > only correct value for the default MSGT_IDENTIFY level.
> > 
> > Right, didn't realize that, ok.
> > 
> > > > Also, the man page should note that identify output is available by 
> > > > -msgl identify=..
> > > 
> > > The patch already includes a mention of that in mplayer.1.
> > 
> > Sorry, I meant it should mention that MORE output is available by -msgl 
> > identify=5 . It doesn't. Other than that, patch OK.
> > 
> > - ods15
> 
> I was going to commit this with an small change on mplayer.1:
> 
> --------------------------8<------------------
> Show file parameters in an easily parseable format.
> Also prints more detailed information about subtitle and audio
> track languages and IDs. If you want even more information
> use -msglevel identify=5.
> ------------------------>8---------------------
> 
> but wile testing I noticed there isnt such a difference, with
> -msglevel identify=5 i get basically the same output, maybe i got
> things wrong somewhere along the line?
> 
> Waiting
> 
>    Reynaldo

Commited.

Ok, i didnt waited too much, just hope the man page entry is right
if it isnt, let me know and I will fix it in no time.

Best regards

	Reynaldo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: Digital signature
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20060424/c2a40793/attachment.pgp>


More information about the MPlayer-dev-eng mailing list