[mpris] Playlist extension observations

[Ian Monroe]

On Fri, Dec 10, 2010 at 09:35, Conor Curran <conor.curran at canonical.com>wrote:

>  Hey folks,
>
> Following what was previously suggested, Bertrand and I  have implemented
> the proposed playlist extension. Here are a list of modifications which I
> think are necessary inorder to make this production ready.
>
>
>    - GetPlaylists should allow for it to be called in an async manner.
>    This can be achieved by passing in a pointer to a PlaylistDetails[]  (
>    PlaylistDetails being a struct of ObjectPath, string (name) and string (icon
>    path)). Or we could add  a new callback ?
>
> A pointer??

Callback makes sense to me.

>
>    -
>    - A new method GetCurrentPlaylist ( also accommodating async usage )
>    should be added. This would be useful for clients that come up and down and
>    hence need to query the server (media player) when starting up to sync
>    properly. If there is no active playlist, it should return a PlaylistDetail
>    with each member set to null ?
>     - A new signal to inform that the current active playlist has changed.
>
>     - This could also cover the situation where the playlist has ended and
>       no other playlist has begun in which case the returned playlist details
>       struct should contain null members.
>
> Thoughts ?
>

Perhaps we should only support async and drop all the sync methods if async
is needed.

I'm wonder why async is needed though? Most players would have that
information on-hand since they have a list of playlists somewhere.

Ian
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mailman.videolan.org/pipermail/mpris/attachments/20101210/6023209e/attachment.html>