[MPlayer-dev-eng] [PATCH] Do not call to th_decode_ycbcr_out() for packets representing a dropped (0-byte) frame.

[Giorgio Vazzana]

2012/1/17 Reimar Döffinger <Reimar.Doeffinger at gmx.de>:
> On Tue, Jan 17, 2012 at 06:34:22PM +0100, Giorgio Vazzana wrote:
>> ps: Not related to this patch, but I've just found two other streams
>> that only seem to work with -demuxer ogg (although ffplay can play
>> them):
>>
>> http://modulix.org:8000/libre.ogg
>> http://modulix.org:8000/savoir.ogg
>
> The fact that ffplay can play them probably is probably
> more a bug of ffplay.

Actually, I think ffplay does the right thing. Look at this sample
I've saved using wget:

http://mywing.altervista.org/tmp/libre.ogg

Using the attached patch it's easy to see that the the first video
packet has the following characteristics:

bytes=4300, granulepos=661456=41341|0, packetno=4, pts=1653.640000

so it's a keyframe, with granulepos != -1, and pts=27m:33s:64. Of
course since it's a live stream we're talking about, we don't want to
wait 27 minutes before start playing the file, so I think ffplay
behaviour is correct.

> If you use ffprobe -show_packets you will see that the
> first video packet has a timestamp of 0 and the next
> of around 1600 seconds.

ffprobe -show_packets for the sample above reports a pts of 0 for the
first video packet, which I think it's wrong, because the correct pts
is 1653.640000

> That should be about the time MPlayer will start to continue
> playing the video part (that it continues to play audio even
> though the timestamps are the same is just because it can't any
> different).
> _______________________________________________
> MPlayer-dev-eng mailing list
> MPlayer-dev-eng at mplayerhq.hu
> https://lists.mplayerhq.hu/mailman/listinfo/mplayer-dev-eng
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0003-theora-show-packets-info.diff
Type: text/x-patch
Size: 1211 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20120118/7d184ba7/attachment.bin>