<div dir="ltr">Awesome! To answer JB's question, I call the method whenever the notification & the widget are updating.<div> <div>Quick note: further testing revealed that mLibVLC.isPlaying() returned some pretty odd results (pressing Next or Previous would randomly return True or False). Let me know if you think I did something wrong.<br></div></div><br><div class="gmail_quote">On Fri, Apr 10, 2015 at 10:20 AM Thomas Guillem <<a href="mailto:thomas@gllm.fr" target="_blank" onclick="window.open('https://mail.google.com/mail/?view=cm&tf=1&to=thomas@gllm.fr&cc=&bcc=&su=&body=','_blank');return false;">thomas@gllm.fr</a>> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
On Thu, Apr 9, 2015, at 10:44, Jean-Baptiste Kempf wrote:<br>
> On 09 Apr, Guillaume wrote :<br>
> > Thanks for your insights. I made the following changes:<br>
> ><br>
> > * If the media type isn't audio, the method is aborted.<br>
> > * It's no longer using the Util methods, as Thomas recommended.<br>
> > * The method doesn't require a Context argument anymore, since the service<br>
> > itself is a Context.<br>
><br>
> Good.<br>
><br>
> Why do you need to broadcast in so many places in the code? 8 times...<br>
<br>
Because the code needs heavy refactoring.<br>
<br>
Your patch is applied in master branch.<br>
Thanks for your contribution.<br>
<br>
><br>
> With my kindest regards,<br>
><br>
> --<br>
> Jean-Baptiste Kempf<br>
> <a href="http://www.jbkempf.com/" target="_blank">http://www.jbkempf.com/</a> - +33 672 704 734<br>
> Sent from my Electronic Device<br>
> ______________________________<u></u><u></u>_________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>l<u></u>istinfo/vlc-devel</a><br>
______________________________<u></u><u></u>_________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" target="_blank">https://mailman.videolan.org/<u></u>l<u></u>istinfo/vlc-devel</a><br>
</blockquote></div></div>