D8942: Make the player status a per-player object in the MPRIS plugin

Matthijs Tijink noreply at phabricator.kde.org
Tue Nov 28 13:06:30 UTC 2017


mtijink updated this revision to Diff 23087.
mtijink added a comment.


  Thanks for the feedback! I incorporated the suggested changes (controlling a player via the new `MprisPlayer` object directly instead of via the plugin and letting the `MprisPlayer` object report correct properties even for Spotify).
  
  The diff is a bit larger this way, but the end result is cleaner.

REPOSITORY
  R225 KDE Connect - Android application

CHANGES SINCE LAST UPDATE
  https://phabricator.kde.org/D8942?vs=22735&id=23087

BRANCH
  mpris-data-per-player

REVISION DETAIL
  https://phabricator.kde.org/D8942

AFFECTED FILES
  src/org/kde/kdeconnect/Plugins/MprisPlugin/MprisActivity.java
  src/org/kde/kdeconnect/Plugins/MprisPlugin/MprisPlugin.java

To: mtijink, #kde_connect
Cc: thomasp
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20171128/da19121b/attachment.html>


More information about the KDEConnect mailing list