[FFmpeg-devel] [PATCHv2 3/4] mpegts: add fix_teletext_pts mpegts demuxer option

[Michael Niedermayer]

On Sun, Aug 04, 2013 at 04:17:22AM +0200, Marton Balint wrote:
> Changes since v1 of the patch:
> - enable option by default
> - add documentation
> - move up PTS override code after PES header parsing, to ensure we use the
>   last PCR before the first packet of the teletext PES packet.
> 
> The option overrides teletext packet PTS and DTS values with the timestamps
> calculated from the PCR of the first program which the teletext stream is part
> of and is not discarded.
> 
> Using the same teletext PID for multiple programs is possible, therefore we
> need some kind of heuristics to know which program PCR we should synchronize
> to. Using the first non-discarded PCR pid among the programs of the teletext
> stream seemed like a good choice.
> 
> The patch does not do PCR interpolation to estimate the PCR of the teltetext
> packet, it just uses the last PCR of the program, which may cause a slight
> error (0.1 sec) in the teletext packet pts-es.
> 
> Based on a patch by Reimar Döffinger.
> http://lists.ffmpeg.org/pipermail/ffmpeg-devel/2012-September/131610.html
> 
> Signed-off-by: Marton Balint <cus at passwd.hu>
> ---
>  doc/demuxers.texi     | 13 +++++++++++++
>  libavformat/mpegts.c  | 47 +++++++++++++++++++++++++++++++++++++++++++++--
>  libavformat/version.h |  2 +-
>  3 files changed, 59 insertions(+), 3 deletions(-)

applied

thanks

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Rewriting code that is poorly written but fully understood is good.
Rewriting code that one doesnt understand is a sign that one is less smart
then the original author, trying to rewrite it will not make it better.
-------------- 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/20130806/043126de/attachment.asc>