<br><br><div class="gmail_quote">On Sat, Mar 10, 2012 at 2:41 PM, 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 Sat, Mar 10, 2012 at 06:26:10AM -0800, Sukrit Sangwan wrote :<br>
<div class="im">> +#define DELAY       20<br>
> +#define DRY_MIX     0.8<br>
</div>> +#define CROSSFEED   0.3<br>
> +#define FEEDBACK    0.3<br>
<br>
Should, some of those, be configurable?<br>
<br></blockquote><div>I have made all of them configurable..<br> <br></div><blockquote class="gmail_quote" style="margin:0pt 0pt 0pt 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
> +    if ( p_filter->fmt_in.audio.i_format != VLC_CODEC_FL32 ||<br>
<br>
> +    if ( p_filter->fmt_in.audio.i_channels != 2 )<br>
<br>
> +    if( unlikely(!p_sys) )<br>
<div class="im"><br>
> +    if( unlikely(!p_sys->pf_begin) )<br>
<br>
</div>Are you using 'if( ' or 'if ( ' ?<br></blockquote><div> </div><div> if( trailing_spaces )<br>{<br>    delete_spaces( );<br>}<br></div></div><br clear="all"><br>-- <br><div>Thanks<br>Sukrit Sangwan</div>

<br>