Hi,<br><br><div class="gmail_quote"><blockquote class="gmail_quote" style="border-left: 1px solid rgb(204, 204, 204); margin: 0pt 0pt 0pt 0.8ex; padding-left: 1ex;"> Well applying non clean code is really a bad idea in the long run.<br>


I would really prefer that the code is reviewed and then applied if clean,<br>
especially with new module API.<br>
<br>
 As a general remark, the patchs used way too much defines where simple functions<br>
or inlines functions would be cleaner and safer.<br>
<br>
 Adding doxygen comment to vlc_sql.h is also needed (at least).<br>
<br>
 I can propose myself to review the patchs once the above remarks are<br>
corrected if you want.<br></blockquote><div>
<br>
I'm just wondering what the current status on this is? I noticed that
there were some commits to vlc-ecp-project.git subsequent to this which
look like they were trying to address the issues raised.<br>
<br>
I am asking mainly because I would like to use the media library for my GSoC
project (media center interface) so am wondering what the best way to
use it would be. If the code is going to be merged soon, then it would be easiest to merge those changes from master into my branch. Otherwise, I'll probably just use apply the patches to my own branch for now.</div>

</div><br>Regards,<br>Barry<br>