[MPlayer-dev-eng] [PATCH] allow changing osd font scale independently from sub font one

Guillaume LECERF foxcore at gmail.com
Wed Dec 26 01:53:49 CET 2007


2007/12/24, Ulion <ulion2002 at gmail.com>:
> The sub_scale seems not work as before.

What do you mean ?
This patch fix the fact that the menu font was rescaled when you
scaled subtitles.
Now, vo_font is scaled by osd_font_scale_factor and sub_font by
text_font_scale_factor (which is the one altered by
mp_property_sub_scale() in command.c).

The previous patch was not correct, here is the correct one.

I also found that this code in vf_menu::config() :

if (force_load_font) {
  force_load_font = 0;
  load_font_ft(width,height,&vo_font,font_name);
}

is only called once.
I suggest the attached patch which loads the font only if vo_font is
null, and not depending on force_load_font because config() won't be
called after force_load_font has been changed.

-- 
Guillaume LECERF
GeeXboX developer - www.geexbox.org
-------------- next part --------------
A non-text attachment was scrubbed...
Name: osd-scale.diff
Type: text/x-diff
Size: 4626 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071226/09f1614b/attachment.diff>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: load_vo_font.diff
Type: text/x-diff
Size: 517 bytes
Desc: not available
URL: <http://lists.mplayerhq.hu/pipermail/mplayer-dev-eng/attachments/20071226/09f1614b/attachment-0001.diff>


More information about the MPlayer-dev-eng mailing list