[MPlayer-dev-eng] [PATCH] Add DVB teletext support

Carl Eugen Hoyos cehoyos at ag.or.at
Thu Oct 29 23:42:33 CET 2009


Reimar Döffinger <Reimar.Doeffinger <at> gmx.de> writes:

> > > > The first patch deals with file name changes and must be preceded by:
> > > >         svn mv stream/tvi_vbi.c libmpcodecs/dec_teletext.c
> > > >         svn cp stream/tv.h libmpcodecs/dec_teletext.h
> > 
> > I will apply this hunk if nobody objects.
> 
> That is the first diff?
> Seems ok.

Applied.

> But I think the TVI -> VBI renaming stuff should be applied shortly after, too.

Hopefully done correctly.

> Most of that is in patch 2, but unfortunately patch 2 also contains stuff like
> adding demuxer->teletext which IMO needs some thinking about or at least a
> proper review, which is hard to do while it is among those huge "cosmetic"
> changes.

Francesco, please resubmit, and Reimar, please review.

Thank you both, Carl Eugen




More information about the MPlayer-dev-eng mailing list