<div class="gmail_quote">On Fri, Dec 10, 2010 at 09:35, Conor Curran <span dir="ltr"><<a 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>A pointer??<br><br>Callback makes sense to me.<br></div><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>
      </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>Ian <br></div></div>