[MPlayer-dev-eng] A recent patch for ass support in mencoder

[Reimar Döffinger]

On Mon, Feb 02, 2009 at 09:43:32PM +0300, Evgeniy Stepanov wrote:
> On Monday 02 February 2009 03:55:38 rvm wrote:
> > El Domingo, 1 de Febrero de 2009 23:13, Eric Appleman escribió:
> > > Hi. I was wondering if the patch listed below was ever considered for
> > > incorporation. I know it may be slightly dated with respect to current
> > > revisions, but I believe it will be are a great template to work from.
> > >
> > > http://smplayer.svn.sourceforge.net/viewvc/smplayer/mplayer-builds/patche
> > >s/ ass_support_for_mencoder_with_embeddedfonts.patch
> >
> > That patch was actually taken from this list:
> > http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/2007-November/054656.ht
> >ml
> >
> > I just did a small modification so the option -embeddedfonts would work
> > with it.
> 
> Well, this patch seems pretty harmless. All questionable functionality is 
> disabled by default. Also, users love it :)
> 
> I'm going to split and apply it in a few days unless anyone objects.

Fine by me, though I'd like to know why the code block starting with
// current_module="read_subtitles_file";
was moved, to me it does not look necessary...