[vlc-devel] [PATCH] AVFormat Demux : Reset PCR for streams initialized later on.

Jai Menon jmenon86 at gmail.com
Tue Jul 27 14:41:00 CEST 2010


On Tue, Jul 27, 2010 at 6:06 PM, Laurent Aimar <fenrir at elivagar.org> wrote:
> Hi,
>
> On Tue, Jul 27, 2010 at 05:04:37PM +0530, Jai Menon wrote:
>> > would you be okay with updating the pcr after receiving a few packets
>> > instead? this way nothing is lost.
>  Doing so force a higher bufferisation and create jitter/delay while reading
> the source stream which isn't nicer I fear (not sure if it creates issue with
> the clock algo). Besides, it has hardcoded value and so seems too much like a
> workaround.

yeah, i agree

>  What's your exact issue?

http://v2v.cc/~j/samples/dvcp_vlc_playback.mov
http://samples.mplayerhq.hu/mov/FCP/DVCPRO.mov
http://samples.mplayerhq.hu/sub/small.divx
etc

>  One thing that isn't nice with the current code is that the pcr is equal
> to the _max_ value of the max values seen per track while it should be the
> _min_ value of the max value seen per tracks. (But fixing it has a big issue
> with ES that terminates before others)
>  Maybe that's your issue?

yeah, and i've tried to incorporate that in the patch too (if the
value is lesser, use that instead).

-- 
Jai Menon



More information about the vlc-devel mailing list