[vlc-devel] [PATCH] MKV: rework the PCR to match what avformat does

Steve Lhomme robux4 at videolabs.io
Fri Mar 13 09:49:08 CET 2015


On Fri, Mar 13, 2015 at 5:50 AM, Rémi Denis-Courmont <remi at remlab.net> wrote:
> Le 2015-03-12 21:23, Steve Lhomme a écrit :
>>
>> Well, there's the case of 0. It's not set because that's also the
>> value of VLC_TS_INVALID. Should they be shifted with VLC_TS_0 ? Will
>> there be issues between the actual pts/dts of the packet if they are
>> not shifted ?
>
>
> PCR, PTS or DTS should always be shifted by VLC_TS_0 if zero is a legal
> value in the stream.
>
> You're welcome to fix all abuses of 0 as VLC_TS_INVALID though. Once that is
> done, you can use INT64_MIN for VLC_TS_INVALID and 0 for VLC_TS_0.


OK, I think it's mostly done in the Matroska demuxer but I'll double
check the code to make sure we don't miss anything.



More information about the vlc-devel mailing list