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

Michael Niedermayer michael at niedermayer.cc
Thu Apr 26 19:45:54 EEST 2018


On Thu, Apr 26, 2018 at 03:15:08AM +0200, Marton Balint wrote:
> 
> 
> On Thu, 26 Apr 2018, Carl Eugen Hoyos wrote:
> 
> >2018-04-25 4:07 GMT+02:00, Aman Gupta <ffmpeg at tmm1.net>:
> >
> >>Processes only teletext pages marked as subtitles, so
> >>depending on the stream it might not produce any output.
> >
> >Shouldn't there be at least an option to output whatever
> >page was requested?
> >
> >+1 for not needing an external dependency.
> 
> Why? Libzvbi is LGPL. We might as well include *that* in the ffmpeg
> codebase...

iam not against including it at all, if its unmaintained outside and
not widely available and someone wants to maintain it in ffmpeg.

I mean if its usefull to us and would without action die, it surely
would make sense ...

some mails in this thread sound a bit in this direction but i do not
know ...



> 
> I thought having a decoder which is inferior in features to an existing one
> or re-inventing yet something again in ffmpeg code are two things we are
> trying to avoid.

Its important that our code is feature complete in relation to what our users
expect.
I do not know how libzvbi compares here to the patch.

also my personal contact with teletext very very long ago didnt give me the
impression that it was complex. This may have changed so my view may be
drawn toward a misconception that this is not that complex ...

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

I am the wisest man alive, for I know one thing, and that is that I know
nothing. -- Socrates
-------------- 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/23e1c5bf/attachment.sig>


More information about the ffmpeg-devel mailing list