thank you for review! patch updated, just change CLIP() to VLC_CLIP()<br><br><div class="gmail_quote">On Wed, Nov 16, 2011 at 12:01 AM, Rafaël Carré <span dir="ltr"><<a href="mailto:funman@videolan.org">funman@videolan.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Le Tue, 15 Nov 2011 15:15:44 +0800,<br>
Can Wu <<a href="mailto:wu.canus@gmail.com">wu.canus@gmail.com</a>> a écrit :<br>
<br>
Hi,<br>
<br>
This patch looks nice, thanks<br>
<br>
However I fear the name 'CLIP' might clash with some header of some library.<br>
<br>
We should use __CLIP like the old macros, or use a prefix like VLC_CLIP<br>
(because _ prefixes are illegal according to C standard)<br>
<span class="HOEnZb"><font color="#888888"><br>
--<br>
Rafaël Carré<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></span></blockquote></div><br><br clear="all"><br>-- <br>wucan<br><br>