On Tue, Feb 09, 2010 at 05:25:37PM +0100, Reimar D?ffinger wrote:
> On Tue, Feb 09, 2010 at 01:02:58PM +0000, Zongyao QU wrote:
> > Zongyao QU <zongyao.qu <at> gmail.com> writes:
> >
> > > How to reproduce?
> > > 1. ./mplayer -ass -font ~/Library/Fonts/msyh.ttf -quiet -slave
> > > -sub-fuzziness 1
> > > {the movie path}
> > > 2. with the slave mode, type
> > > set_property sub_scale 3
> > >
> > > 3. sub_load "the sub path"
> >
> > It seams that there is a bug in command.c
> >
> > when use "set_property sub_scale X",
> > the coeff of ass and non-ass will be updated both.
>
> I think you're looking at the wrong place, please try attached patch.
Sorry, after testing properly, I think this might be the way to go:
Index: libass/ass_mp.c
===================================================================
--- libass/ass_mp.c (revision 30526)
+++ libass/ass_mp.c (working copy)
@@ -93,7 +93,7 @@
style->FontName = (font_fontconfig >= 0 && sub_font_name) ? strdup(sub_font_name) : (font_fontconfig >= 0 && font_name) ? strdup(font_name) : strdup("Sans");
style->treat_fontname_as_pattern = 1;
- fs = track->PlayResY * text_font_scale_factor / 100.;
+ fs = track->PlayResY / 100.;
// approximate autoscale coefficients
if (subtitle_autoscale == 2)
fs *= 1.3;
