Hi,<div><br></div><div>Everything is fixed. Patch attached.<br><br><div class="gmail_quote">On Tue, Nov 10, 2009 at 4:36 AM, Laurent Aimar <span dir="ltr"><<a href="mailto:fenrir@via.ecp.fr">fenrir@via.ecp.fr</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Hi,<br>
<div class="im"><br>
On Fri, Nov 06, 2009, Srikanth Raju wrote:<br>
>    I did this. But it seems to make the include a lot heavier. Not too heavy<br>
>    though.<br>
</div> If it is a problem, they could be moved to the core (later is fine).<br>
<br>
> +#include <assert.h><br>
 Sadly, it is a bad idea to include <assert.h> in a public header, and<br>
so you should remove it.<br>
<br>
> +typedef struct sql_stmt_sys_t sql_stmt_sys_t;<br>
Useless.<br>
<br>
>      /** Rollback transaction */<br>
>      int (*pf_rollback) ( sql_t* );<br>
In the current commited file, it is void and not int. It prevents me to<br>
apply the patch.<br>
<br>
> +struct sql_stmt_t;<br>
Useless.<br>
<br>
>  /*****************************************************************************<br>
>   * SQL Function headers<br>
> @@ -249,6 +305,267 @@ static inline int sql_RollbackTransaction( sql_t *p_sql )<br>
Same here, void instead of it.<br>
<br>
> +static inline int sql_GetColumnInteger( sql_t* p_sql, sql_stmt_t* p_stmt,<br>
> +        int i_col, int* i_res )<br>
> +{<br>
> +    assert( i_res != NULL );<br>
 You will need to removes the assert()<br>
<br>
 Otherwise, the patch is fine (I would have applied it, but because of<br>
rollback() and assert(), I prefered not to modify your patch myself).<br>
<br>
Regards,<br>
<div><div></div><div class="h5"><br>
--<br>
fenrir<br>
<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>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>Regards,<br>Srikanth Raju<br>
</div>