[MPlayer-users] [bug report] subtitle size inconsistency between?pre-load sub and runtime-load sub

[Reimar Döffinger]

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;