<br><br><div class="gmail_quote">On Sun, Mar 20, 2011 at 5:11 PM, Jean-Baptiste Kempf <span dir="ltr"><<a href="mailto:jb@videolan.org" target="_blank">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 Sun, Mar 20, 2011 at 03:57:46PM +0530, Pankaj yadav wrote :<br>
> + * Copyright (C) 20011 the VideoLAN team<br>
18k years from now? NICE<br></blockquote><div>Rectified </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +    float f_level;<br>
<br>
What is this level?<br></blockquote><div>Removed.We should use a hard-coded value 0.7 as done by m*****r and many others. </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">

<br>
> +    unsigned i_channels;<br>
<br>
You only write in this, not use it.<br></blockquote><div>i have used i_channels.Should i not use it?</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +        msg_Warn( p_filter, "bad input or output format" );<br>
Please more explanation<br>
<br></blockquote><div>Done </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> +     p_sys->f_level = var_CreateGetFloat( p_filter->p_parent, "karaoke-level" );<br>
No tabs.<br></blockquote><div>Removed </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +     p_sys->f_level = var_GetFloat( p_filter->p_parent, "karaoke-level" );<br>
Use inherit*<br></blockquote><div>Removed </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
> +    /* Calculate the average power level on this buffer */<br>
> +    for( i = 0 ; i < i_samples; i++ )<br>
Declare i here<br>
<br></blockquote><div>Done </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
> +    {<br>
> +        p_out[0] = (p_in[0] - p_in[1]) * p_sys->f_level;<br>
> +        p_out[1] = p_out[0];<br>
> +        p_in += i_channels;<br>
> +        p_out += i_channels;<br>
> +    }<br>
> +<br>
> +    return p_in_buf;<br>
Is that all? Ain't there something missing?<br></blockquote><div>Yes.Tested on my windows computer.</div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<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><br>
+33 672 704 734<br>
</font><div><div></div><div>_______________________________________________<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Pankaj Yadav<br><a href="http://people.videolan.org/~pk" target="_blank">http://people.videolan.org/~pk</a><br>Open Source Enthusiast<br>