<div dir="ltr"><br><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Mar 4, 2014 at 8:17 AM, Subodh Shrestha <span dir="ltr"><<a href="mailto:forsubodh@gmail.com" target="_blank">forsubodh@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><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><div class=""><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><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><div><br></div><div>I would like to clarify this a little more.  If you look at /modules/text_renderer/freetype.c, GetFontSize() reads the "freetype-rel-fontsize" config variable and uses the value obtained to set the subtitle fontsize. I'm following a similar idea of using the same config variable to set the fontsize.<br>
<br></div><div>However, I do not want to change the value of the "freetype-rel-fontsize" in the config file permanently because then the config file will get out of sync with the subtitle preference dialog which only has five size settings. Hence I'm using var_Create to create a vlc variable of the same name which will be used during the vlc session if subtitle fontsize is being changed using hotkeys. And when vlc is restarted, config file settings will match that of one of the subtitle preference sizes (since I'm not changing the config file settings).<br>
<br></div><div>Please let me know if this doesn't sound right to you. I still don't understand what you mean by the correct way.<br></div><div><br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div dir="ltr"><div><div>
</div><div class=""><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><div>It's the module to associate with the freetype-rel-fontsize config variable when I retrieve it above.</div><div class=""><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>
</div>
</blockquote></div><br></div></div>