<div class="gmail_quote">2009/7/30 Rémi Denis-Courmont <span dir="ltr"><<a href="mailto:remi@remlab.net">remi@remlab.net</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;">

(...)<br>
> +        playlist_item_t *p_target = 0;<br>
<br>
NULL! Implicit int to pointers is ugly and can trigger type size mismatch if<br>
sizeof(int) != sizeof(intptr_t).<br>
<br>
(...)<br>
> +                int i;<br>
<div class="im">> +                for( i = 0 ; i< p_parent->i_children ; i++ )<br>
<br>
</div>You could move i into the loop, I guess.<font color="#888888"><br>
</font></blockquote></div><br><br>I agree with what you proposed. I've been a lot in C++ lately so got used to 0 instead of NULL. As for variable declaration inside "for" statement, I followed examples that I saw around in the rest of the code.<br>
<br>Would you like me to send corrected patch again or will you do it manually?<br><br>Kind regards!<br>