<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN">
<html>
<head>
<meta content="text/html; charset=ISO-8859-1"
http-equiv="Content-Type">
</head>
<body text="#000000" bgcolor="#ffffff">
Hi Ian,<br>
<br>
<br>
On 10/12/10 15:40, Ian Monroe wrote:
<blockquote
cite="mid:AANLkTinVQVtZMrZKTC-6wjOK=E3OP0-OgUoiMgfBO_d9@mail.gmail.com"
type="cite">
<div class="gmail_quote">On Fri, Dec 10, 2010 at 09:35, Conor
Curran <span dir="ltr"><<a moz-do-not-send="true"
href="mailto:conor.curran@canonical.com">conor.curran@canonical.com</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
0.8ex; border-left: 1px solid rgb(204, 204, 204);
padding-left: 1ex;">
<div text="#000000" bgcolor="#ffffff"> Hey folks,<br>
<br>
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.<br>
<br>
<ul>
<li>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 ?<br>
</li>
</ul>
</div>
</blockquote>
<div><br>
<br>
Callback makes sense to me.<br>
</div>
</div>
</blockquote>
<br>
Good to hear. <br>
<br>
<blockquote
cite="mid:AANLkTinVQVtZMrZKTC-6wjOK=E3OP0-OgUoiMgfBO_d9@mail.gmail.com"
type="cite">
<div class="gmail_quote">
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt
0.8ex; border-left: 1px solid rgb(204, 204, 204);
padding-left: 1ex;">
<div text="#000000" bgcolor="#ffffff">
<ul>
<li> <br>
</li>
<li>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 ? <br>
</li>
<li>A new signal to inform that the current active
playlist has changed. <br>
</li>
<ul>
<li>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.</li>
</ul>
</ul>
Thoughts ?<br>
</div>
</blockquote>
<div><br>
Perhaps we should only support async and drop all the sync
methods if async is needed.<br>
<br>
I'm wonder why async is needed though? Most players would have
that information on-hand since they have a list of playlists
somewhere.<br>
<br>
</div>
</div>
</blockquote>
<br>
I would prefer to call things asynchronously just in case anything
blocks. <br>
It should be fine synchronously but I would rather use a more fool
proof approach. <br>
<br>
Conor<br>
<br>
</body>
</html>