<div dir="ltr">Thanks for your review<br><br>About the tooltip and the suffix, the subsdelay filter has 3 calculation modes which dictates different values<br><br>in short its<br>- absolute delay: new_delay = source_delay + factor<br>
- (default) relative to source delay: new_delay = source_delay * factor<br>- relative to source content: new_delay = f(source_text) * factor<br><br>In the first mode the factor is actually seconds but in the other two it has no units so I can't define a suffix<br>
The same thing about the tip, it's hard to explain without considering the mode<br><br>There are 2 options<br>- change the tip/suffix according to the mode<br>- assume that the default mode hasn't been changed<br>
<br>any preference/suggestions?<br><br><br><div class="gmail_quote">On Wed, May 11, 2011 at 2:11 AM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org">jb@videolan.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">
On Tue, May 10, 2011 at 03:05:01PM +0300, Yuval Tze wrote :<br>
<div class="im">> - add a spin box to the synchronizations tab that enables the subsdelay filter when its value is changed and disables it when its value is set to 0.<br>
<br>
</div>This lacks setToolTips and other niceties to understand what it actually<br>
do...<br>
<div class="im"><br>
> - make the private function ExtVideo::ChangeVFiltersString accessible to other local panels.<br>
</div>OK-ish...<br>
<div class="im"><br>
> -        p_vout = THEMIM->getVout();<br>
> +        vout_thread_t *p_vout = THEMIM->getVout();<br>
<br>
</div>Then you should remove p_vout from the class member (and fix the small<br>
issue that will ensue...)<br>
<div class="im"><br>
> +    subDurationSpin = new QDoubleSpinBox;<br>
> +    subDurationSpin->setAlignment( Qt::AlignRight|Qt::AlignTrailing|Qt::AlignVCenter );<br>
> +    subDurationSpin->setDecimals( 3 );<br>
> +    subDurationSpin->setMinimum( 0 );<br>
> +    subDurationSpin->setMaximum( 20 );<br>
> +    subDurationSpin->setSingleStep( 0.2 );<br>
> +    subsLayout->addWidget( subDurationSpin, 2, 2, 1, 1 );<br>
</div>No suffix?<br>
<div class="im"><br>
> +    vlc_value_t val;<br>
> +<br>
> +    val.f_float = f_factor;<br>
> +    if( var_SetChecked( p_intf->p_libvlc, CFG_SUBSDELAY_FACTOR, VLC_VAR_FLOAT, val ) )<br>
> +    {<br>
> +        var_Create( p_intf->p_libvlc, CFG_SUBSDELAY_FACTOR, VLC_VAR_FLOAT | VLC_VAR_DOINHERIT );<br>
> +        var_SetFloat( p_intf->p_libvlc, CFG_SUBSDELAY_FACTOR, f_factor );<br>
> +    }<br>
</div>Are you sure that you need all that?<br>
If so, you need to document it...<br>
<div class="im"><br>
> +double SyncControls::subsdelayGetFactor()<br>
> +{<br>
> +    vlc_value_t val;<br>
> +<br>
> +    if( !var_GetChecked( p_intf->p_libvlc, CFG_SUBSDELAY_FACTOR, VLC_VAR_FLOAT, &val ) )<br>
> +    {<br>
> +        return val.f_float;<br>
> +    }<br>
</div>I am confused here the same...<br>
<br>
Best Regards,<br>
<font color="#888888"><br>
--<br>
Jean-Baptiste Kempf<br>
<a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - +33 672 704 734<br>
Sent from my Electronic Device<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="http://mailman.videolan.org/listinfo/vlc-devel" target="_blank">http://mailman.videolan.org/listinfo/vlc-devel</a><br>
</font></blockquote></div><br></div>