[MPlayer-dev-eng] Re: [PATCH] SSA/ASS subtitles support

Dominik 'Rathann' Mierzejewski dominik at rangers.eu.org
Sun Jun 18 19:29:06 CEST 2006


On Sunday, 18 June 2006 at 15:38, Rich Felker wrote:
> On Sun, Jun 18, 2006 at 02:33:13AM +0400, Evgeniy Stepanov wrote:
[...]
> > > > +/**
> > > > + * \brief Process embedded matroska font. Saves it to ~/.mplayer/fonts.
> > >
> > > I do not like this at all.
> > > This feature might be misused in many ways, one I can think of is leaving
> > > a track record of which films somebody has been watching.
> > > The fact that it is disabled by default might make it acceptable though.
> > > I prefer if there was a way to use embedded fonts with at most using
> > > temporary files created via tmpfile instead (hopefully this function is
> > > portable enough).
> > 
> > Freetype can read fonts directly from memory, but fontconfig can't :(
> 
> Is there a reason we need fontconfig? I hate it...

But not everyone else on the planet does. ;)

> > I like to have a cache of fonts. What about creating persistent files 
> > with -embeddedfonts and temporary ones otherwise ? Do we need a mode when no 
> > files will be created at all ?
> 
> Yes absolutely. Creating files is a very very bad thing. It will not
> work on read-only filesystems (think live cd with MPlayer and movie
> files) and it leads to bugs and security issues more often than not.
> 
> Why not just bypass fontconfig? I don't even have fontconfig and I'm
> quite happy with the way MPlayer works.

I'm quite happy with fontconfig. I don't have to put path to the font file
in MPlayer config. I don't even have to know which file corresponds to each
font. So please don't insist on taking away that choice.

Regards,
R.

-- 
MPlayer developer and RPMs maintainer: http://rpm.greysector.net/mplayer/
There should be a science of discontent. People need hard times and
oppression to develop psychic muscles.
	-- from "Collected Sayings of Muad'Dib" by the Princess Irulan



More information about the MPlayer-dev-eng mailing list