[vlc-devel] [PATCH] SQL Media Library

Barry Wardell barry.wardell at gmail.com
Fri Jun 12 01:31:11 CEST 2009


Hi,

 Well applying non clean code is really a bad idea in the long run.
> I would really prefer that the code is reviewed and then applied if clean,
> especially with new module API.
>
>  As a general remark, the patchs used way too much defines where simple
> functions
> or inlines functions would be cleaner and safer.
>
>  Adding doxygen comment to vlc_sql.h is also needed (at least).
>
>  I can propose myself to review the patchs once the above remarks are
> corrected if you want.
>

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.

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.

Regards,
Barry
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20090612/fa0ce888/attachment.html>


More information about the vlc-devel mailing list