[MPlayer-dev-eng] [PATCH] ASF: Use play_duration instead of send_duration

[Zuxy Meng]

Hi,

2007/5/24, Compn <tempn at twmi.rr.com>:
> On Thu, 24 May 2007 09:33:43 +0800, Zuxy Meng scribed:
>
> >Hi,
> >
> >2007/5/24, Compn <tempn at twmi.rr.com>:
> >> On Wed, 23 May 2007 12:39:59 +0800, Zuxy Meng scribed:
> >>
> >> >Hi,
> >> >
> >> >Currently the asf's movielength is obtained thru the send time field
> >> >in asf file header, while the correct one to use is play time
> >> >substracted by preroll. Send time should be ignored by a demuxer
> >> >although most encoders will set it to the same value as play time. I
> >> >happened to encounter a clip whose send time isn't properly set,
> >> >leading to incorrect OSD display.
> >> >
> >> >Libavformat demuxed the clip correctly, because although it uses
> >> >send time, libavformat reads the two fields in a reversed order
> >> >(play time appears first in the file header, but libavformat reads
> >> >it as send time).
> >>
> >>
> >> dont forget to update the changelog when you apply this :)
> >
> >Ah...is this trivial fix worth it?
> >
>
> yes, it fixes a bug that some users have been having...
> i've been telling people on irc to use -demuxer lavf.

I see. Can u point me a sample? The asf format itself as well as the
libmpdemux implementation is quite a mess and I'm not sure if this is
the only cause why mplayer has reported incorrect length.

>
> Index: Changelog
> ===================================================================
> --- Changelog   (revision 23378)
> +++ Changelog   (working copy)
> @@ -35,6 +35,7 @@
>     * support for EVO demuxing
>     * support -noidx with -demuxer lavf
>     * support for channel navigation with PVR input
> +    * ASF/WMV demuxer should report correct length now
>
>     Streaming:
>     * authentication for Real RTSP streams


-- 
Zuxy
Beauty is truth,
While truth is beauty.
PGP KeyID: E8555ED6