RE: [xine-devel] patch for closed captioning now available

[Miguel F. <mi...@ce...>]

On Fri, 2002-01-04 at 19:02, Siggi Langauf wrote:
> On Fri, 4 Jan 2002, Heiko Schaefer wrote:
> 
> > miguel's proposition to put it in a package of it's own sounds good to me.
> > after all, xine's libmpeg2 is a variant of michel lespinasse's (and before
> > that aaron's) libmpeg2 - i guess it would make sense to keep the
> > difference between xine's version and michel's version as small as
> > reasonably possible.
> 
> IMHO, it's a good idea to keep the differences to a minimum, but why don't
> we ask Michel to add CC support to libmpeg2? It's a standard mpeg2
> feature, anyway...
> The downside is that it will require an API extension for libmpeg2,
> though.

I'm not sure about CC being standard mpeg2 feature: it's stored on mpeg2
userdata and i think the docs i have about the stardard don't mention
it. specs says that userdata may be safely ignored by the decoder.

So if Michel is going to extend libmpeg2 api i guess he would have to
add some kind of callback for "pluggable" userdata decoding.

Anyway, Christian's code is weakly tied to libmpeg2 but font handling
and other stuff must be very xine dependent. This is just my impression
from browsing the code though.

regards,

Miguel