[FFmpeg-devel] [PATCH] mpegts: do not set pts for missing dts in video streams

[Michael Niedermayer]

On Wed, Dec 04, 2013 at 02:58:06AM +0100, Michael Niedermayer wrote:
> Hi
> 
> On Tue, Dec 03, 2013 at 10:03:04AM +0100, fernetmenta at online.de wrote:
> > From: Rainer Hochecker <fernetmenta at online.de>
> > 
> > ---
> >  libavformat/mpegts.c | 5 ++++-
> >  1 file changed, 4 insertions(+), 1 deletion(-)
> 
> applied

patch reverted
i had checked the spec before appying it but ive missed or
misinterpreted it and thought the dts=pts rule only applied to
mpeg1/2

For reference:
A PTS may only be present in a Rec. ITU-T H.222.0 | ISO/IEC 13818-1 video or audio elementary stream PES packet
header if the first byte of a picture start code or the first byte of an audio access unit is contained in the PES packet.
A decoding_timestamp (DTS) shall appear in a PES packet header if and only if the following two conditions are met:
              a PTS is present in the PES packet header;
              the decoding time differs from the presentation time.

Can you check if reverting this has any ill side effects ?
ive tried a few files and didnt notice a problem but iam not sure
which file the original patch was intended to fix.
Can you check ?

Thanks & sorry for the mess

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

DNS cache poisoning attacks, popular search engine, Google internet authority
dont be evil, please
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 198 bytes
Desc: Digital signature
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140302/09c278e2/attachment.asc>