[vlc-devel] [PATCH] [Android] Implement Scrobbling compatibility (#11904)
Thomas Guillem
thomas at gllm.fr
Fri Apr 10 10:20:01 CEST 2015
On Thu, Apr 9, 2015, at 10:44, Jean-Baptiste Kempf wrote:
> On 09 Apr, Guillaume wrote :
> > Thanks for your insights. I made the following changes:
> >
> > * If the media type isn't audio, the method is aborted.
> > * It's no longer using the Util methods, as Thomas recommended.
> > * The method doesn't require a Context argument anymore, since the service
> > itself is a Context.
>
> Good.
>
> Why do you need to broadcast in so many places in the code? 8 times...
Because the code needs heavy refactoring.
Your patch is applied in master branch.
Thanks for your contribution.
>
> With my kindest regards,
>
> --
> Jean-Baptiste Kempf
> http://www.jbkempf.com/ - +33 672 704 734
> Sent from my Electronic Device
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel
More information about the vlc-devel
mailing list