[MPlayer-dev-eng] New -subfont option

Evgeniy Stepanov eugeni.stepanov at gmail.com
Tue May 8 00:54:59 CEST 2007


On Tuesday 08 May 2007, Benjamin Zores wrote:
> Evgeniy Stepanov a écrit :
> > On Monday 07 May 2007, Benjamin Zores wrote:
> > [...]
> >
> >> Attached a second revision of the patch, with Reimar and Evgeniy
> >> comments.
> >
> > No, you did not.
>
> Working on too many things at a time is a bad idea.
> Hope it's ok now ;-)

This way font_name is ignored even is sub_font_name is NULL, which is not 
consistent with the rest of the patch. I'd prefer something like this 
(attached).

And the new option should be added to the man page.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 1.patch
Type: text/x-diff
Size: 1396 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20070508/858acd53/attachment.patch>


More information about the MPlayer-dev-eng mailing list