[FFmpeg-devel] [PATCHv2 3/4] mpegts: add fix_teletext_pts mpegts demuxer option
michaelni at gmx.at
Tue Aug 6 02:56:19 CEST 2013
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.
> 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(-)
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
Size: 198 bytes
Desc: Digital signature
More information about the ffmpeg-devel