Hey,<br>Sorry about the delay. Changes made and attached.<br><br><div class="gmail_quote">2009/4/17 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:rem@videolan.org">rem@videolan.org</a>></span><br><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;">
        Hello,<br>
<br>
Le mardi 7 avril 2009, Srikanth Raju a écrit :<br>
<div class="im">> I've attached a basic chorus/flanger audio filter. Need some feedback<br>
> for it.<br>
<br>
</div>I'm not very knowledgeable on audio filtering. However, you should:<br>
 - remove dead debugging code, especially with swear words,<br>
 - not include header files you really don't need (errno, ctype, signal)<br>
 - not alter the filter object when Open() returns VLC_EGENERIC.<br>
 - not leak p_sys when Open() fails.<br>
 - remove the useless if(p_sys) in Close().<br>
<br>
Best regards,<br>
<font color="#888888"><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.net/" target="_blank">http://www.remlab.net/</a><br>
</font></blockquote></div><br><br clear="all"><br>-- <br>Regards,<br>Srikanth Raju<br>