[vlc-devel] Media library Include file and Core

[Srikanth Raju]

On Tue, Jun 22, 2010 at 5:03 PM, Jakob Leben <jakob.leben at gmail.com> wrote:

> On Mon, Jun 21, 2010 at 7:53 PM, Srikanth Raju <srikiraju at gmail.com>wrote:
>
>> Hello,
>> I have attached two patches. I will merge these into master by Wednesday
>> if nobody has any problems with them. These are the include files and some
>> minor changes to the core.
>>
>
>
> Hi, Srikanth,
>
> Maybe I'm a bit late, but I have a couple of conceptual questions about
> this, so please enlighten me:
>
> I was wondering what is the reasoning behind adding all the information
> fields like artist, album, track number, etc. to ml_media_t instead of
> re-using vlc_meta_t (or developing it, if it is not sufficient) ?
> For that matter, couldn't an input_item_t be used on the place of
> ml_media_t?
>
>
This was much argued over last year. We decided that input_item_t and
vlc_meta_t will represent information that can be obtained from the file
directly and ml_media_t will be what is stored in the sql database. There
are numerous reasons for this
* input_item_t can represent streams, read only files, etc, and editing
media information and storing it may not always be possible. In cases where
the file is modifiable, formats may not support specific meta data. Handling
this would be overly complicated, cumbersome, with all sorts of format
specific hacks needed.
* the current vlc_meta_t scheme doesn't support all the metadata we want or
would like in the future. Having a new structure that is free from the
format will make the data structure much more flexible.
* We don't want to mess with any current functionality, so it's best to keep
those working functions there, for now.
* There is a lot of metadata, and this might increase. We don't want to load
all that information if the user isn't using the media library.

Or is the idea for development just the opposite, to remove that information
> from input_item_t, once the new ML is done?


I suspect this is what will eventually happen, only for the metadata in
input_item_t ofcourse.


> Cheers,
>
> Jakob
>
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> http://mailman.videolan.org/listinfo/vlc-devel
>
>


-- 
Regards,
Srikanth Raju
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/vlc-devel/attachments/20100622/83e70346/attachment.html>