[vlc-devel] [PATCH] libvlc: expand media player API to retrieve information about all available titles of the currently playing media item

[Rémi Denis-Courmont]

Le 2015-03-26 16:24, Felix Paul Kühne a écrit :
> Hello Rémi,
>
>> On 26 Mar 2015, at 14:14, Rémi Denis-Courmont <remi at remlab.net> 
>> wrote:
>>>
>>> /**
>>> + * Get the full description of available titles
>>> + *
>>> + * \param p_mi the media player *
>>> + * \return list containing full description of available titles
>>> + * \version LibVLC 3.0.0 and later.
>>> + */
>>> +LIBVLC_API libvlc_title_description_t *
>>> +        libvlc_media_player_get_title_descriptions(
>>> libvlc_media_player_t *p_mi );
>>
>> Why do you return a list rather than a table?
>
> Why not? :)
>
> I thought it would be a nicer approach and it matches many other
> libvlc functions.

How is a linked list nicer than a table of pointers?

>>> +        case INPUT_GET_FULL_TITLE_INFO:
>>
>> Why do you need a new control and why is it only used by LibVLC?
>
> Because of your feedback from early February.

Sure you need a single control to fetch all the data atomically. Still 
why do you introduce a new control instead of extending 
INPUT_GET_TITLE_INFO?

-- 
Rémi Denis-Courmont