<div dir="ltr"><div dir="ltr">Hi,<div><br></div><div>The --sub-text-scale command-line switch was already an integer in VLC 3.x, and internally uses var_CreateInteger, var_GetInteger and so on.</div><div><br></div><div>From the review comments so far:</div><div>- you would prefer float rather than int for the proposed new LibVLC API method</div><div>- you do not want to have mismatched float for the LibVLC API and int for the corresponding VLC command line option</div><div><br></div><div>So does that mean for this patch to be acceptable you would want it to include a change to the preexisting --sub-text-scale switch to make that switch a float instead of an int?</div><div><br></div><div>Regards,</div><div><br></div></div><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, 20 Apr 2020 at 14:56, Rémi Denis-Courmont <<a href="mailto:remi@remlab.net">remi@remlab.net</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">Le maanantaina 20. huhtikuuta 2020, 15.34.44 EEST Thomas Guillem a écrit :<br>
> Hello, thanks for this patch and for responding to our review very quickly.<br>
> <br>
> I don't have any preferences regarding Integer of float for scale. But do we<br>
> really want to have float for libvlc and int for vlc command line options ?<br>
<br>
Of course not. We use float everywhere except UI, e.g. --rate or --zoom.<br>
<br>
<br>
-- <br>
雷米‧德尼-库尔蒙<br>
<a href="http://www.remlab.net/" rel="noreferrer" target="_blank">http://www.remlab.net/</a><br>
Rémi Denis-Courmont<br>
<br>
<br>
<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div><div><br></div></div>