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

[Zongyao QU]

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.

---------------------------------------------------
1817a1817,1820
>             } else {
> #endif
> 				text_font_scale_factor = *(float *) arg;
> #ifdef CONFIG_ASS
1820d1822
<             text_font_scale_factor = *(float *) arg;
1828c1830
<                   (action == M_PROPERTY_STEP_UP ? 1.0 : -1.0);
---
> 				(action == M_PROPERTY_STEP_UP ? 1.0 : -1.0);
1830a1833,1838
>             } else {
> #endif
> 				text_font_scale_factor += (arg ? *(float *) arg : 0.1)*
> 				(action == M_PROPERTY_STEP_UP ? 1.0 : -1.0);
> 				M_PROPERTY_CLAMP(prop, text_font_scale_factor);
> #ifdef CONFIG_ASS
1833,1835d1840
<             text_font_scale_factor += (arg ? *(float *) arg : 0.1)*
<               (action == M_PROPERTY_STEP_UP ? 1.0 : -1.0);
<             M_PROPERTY_CLAMP(prop, text_font_scale_factor);
2986a2992
> 		    }
2988d2993
< 	    }
---------------------------------------------------