[vlc-devel] [PATCH 2/4] Media library CRUD operations

[Jean-Baptiste Kempf]

On Mon, Oct 18, 2010 at 01:10:11AM +0530, Srikanth Raju wrote :
> Patch 2, straight forward stuff.
> + * Authors: Antoine Lejeune <phytos at videolan.org>
> + *          Jean-Philippe Andr?? <jpeg at videolan.org>
> + *          R??mi Duraffort <ivoire at videolan.org>
> + *          Adrien Maglo <magsoft at videolan.org>
> + *          Srikanth Raju <srikiraju at gmail dot com>
Encoding issue? Or is it my mail client?

> +    switch( selected_type )
Can't some part be factored?

> +    if( p_media->psz_uri && p_media->i_id )
Same here?
At least some macros...

Best Regards,

-- 
Jean-Baptiste Kempf
http://www.jbkempf.com/
+33 672 704 734