[MPlayer-dev-eng] [PATCH] redundant output for ASF

[Diego Biurrun]

D Richard Felker III writes:
> On Tue, Oct 19, 2004 at 01:30:46AM +0200, Diego Biurrun wrote:
> > 
> > I noticed some time ago that ASF movies produce redundant output like
> > this:
> > 
> > VIDEO:  [WMV3]  320x240  24bpp
> > VIDEO:  [WMV3]  320x240  24bpp  1000.000 fps    0.0 kbps ( 0.0 kbyte/s)
> > 
> > I tracked this down to libmpdemux/demuxer.c where this info gets
> > printed once for ASF and then for all video types.  This patch removes
> > the redundant mp_msg call.
> > 
> > Please tell me if I am overlooking something, otherwise I will apply
> > this soon.
> 
> looks ok. i added the 'redundant' one printing the second copy of the
> line because most demuxers suck too much to print that info and it's
> (imnsho) the most important line in mplayer's output. so removing the
> asf-specific one should be perfect.

That line is very useful indeed compared to some of the other stuff
that gets printed on a non-verbose run.

Applied.

Diego