[FFmpeg-devel] [PATCH] avcodec: add minimal teletext subtitle decoder

Michael Niedermayer michael at niedermayer.cc
Thu Apr 26 02:53:25 EEST 2018


On Wed, Apr 25, 2018 at 09:32:43PM +0200, Marton Balint wrote:
> 
> 
> On Tue, 24 Apr 2018, Aman Gupta wrote:
> 
> >From: Aman Gupta <aman at tmm1.net>
> >
> >Based largely on VLC's modules/codec/telx.c.
> >
> >Processes only teletext pages marked as subtitles, so depending
> >on the stream it might not produce any output.
> >
> >Subtitles are rendered directly to ASS, with support for background
> >colors and a best-effort at screen positioning. The ASS packets
> >are emitted in real time (similar to ccaption_dec's real_time
> >option), with -1 durations. The decoder expects that the player
> >will remove all existing subtitles whenever a new packet arrives.
> >
> >The teletext clear command is implemented using an empty subtitle,
> >which removes existing subtitles but does not render anything new.
> 
> Why don't you add the ASS styling functionality to the existing
> libzvbi-teletextdec.c instead? You're duplicating lot's of code and doing
> teletext parsing which is hard to do properly so using libzvbi for it seems
> a better choice.

I dont know about amans arguments or reasons but
I think avoiding an external dependancy would be an argument not to make
the code depandant on libzvbi

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

Everything should be made as simple as possible, but not simpler.
-- Albert Einstein
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <http://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20180426/09b411a6/attachment.sig>


More information about the ffmpeg-devel mailing list