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

[Compn]

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.


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