<div dir="ltr">Please see patch 2 ( uploaded with patch 1) for the cleanup. I will upload a new clean patch as per Felix's suggestion.<div><br></div><div><br>On Tuesday, March 4, 2014, Jean-Baptiste Kempf <<a href="mailto:jb@videolan.org" target="_blank">jb@videolan.org</a>> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
On 04 Mar, Subodh Shrestha wrote :<br>
> Ctrl+a increases subtitle fontsize<br>
> Ctrl+Shift+a decreases subtitle fontsize<br>
<br>
Please don't use Ctrl. Use Alt.<br>
<br>
> -<br>
> +<br>
<br>
Empty not-wanted change.<br>
<br>
> +        case ACTIONID_FONTSIZE_UP:<br>
> +            if ( p_filter )<br>
> +            {<br>
> +                int rel_fontsize = var_InheritInteger( p_filter, "freetype-rel-fontsize");<br>
> +                msg_Err(p_filter, "Old Relative Fontsize:%i", rel_fontsize);<br>
> +                var_Create( p_filter, "freetype-rel-fontsize", VLC_VAR_INTEGER);<br>
> +                var_SetInteger( p_filter, "freetype-rel-fontsize", --rel_fontsize);<br>
<br>
I really doubt this a correct way.<br>
Create?<br></blockquote><div><br></div><div>I don't understand your comment here. If I don't use var_create, I can't seem to update the freetype-rel-fontsize config variable. I will look to see if there's a better way.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">
<br>
> +    //SS>><br>
> +    filter_t *p_filter = (filter_t *)libvlc;<br>
> +    //SS<<<br>
<br>
What is that?<br></blockquote><div><br></div><div>It's the module to associate with the freetype-rel-fontsize config variable when I retrieve it above.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
> -<br>
> +<br>
<br>
See above.<br>
<br>
> +        //SS<br>
> +        msg_Err(p_filter, "here");<br>
<br>
...<br>
<br>
<br>
This patch needs a lot of cleaning, I'm sorry... </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<br>
With my kindest regards,<br>
<br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - <a href="tel:%2B33%20672%20704%20734" value="+33672704734" target="_blank">+33 672 704 734</a><br>
Sent from my Electronic Device<br>
</blockquote></div>
</div>