Attached the patches addressing the review comments. Had put the fixes into original patch. <div><br></div><div>-Dharani<br><br><div class="gmail_quote">2011/4/10 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</a>></span><br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Le samedi 9 avril 2011 21:41:40 dharani prabhu, vous avez écrit :<br>
<div class="im">> Hi,<br>
><br>
>     I have attached the patch addressing the review comments mentioned<br>
> below. I have developed the video filter based on virtualdub. Pls see<br>
> <a href="http://neuron2.net/deflick/flick.html" target="_blank">http://neuron2.net/deflick/flick.html</a> for more details about the video<br>
> filter and sample videos<br>
<br>
</div>Please squash the fixes into the original patch. And there is no point<br>
checking that a pointer is non-NULL before calling free().<br>
<font color="#888888"><br>
--<br>
Rémi Denis-Courmont<br>
<a href="http://www.remlab.info/" target="_blank">http://www.remlab.info/</a><br>
<a href="http://fi.linkedin.com/in/remidenis" target="_blank">http://fi.linkedin.com/in/remidenis</a><br>
</font><div><div></div><div class="h5">_______________________________________________<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></div>