Review Request 128271: MPRIS: Send info about whether certain actions are allowed. (KDE)

Aleix Pol Gonzalez aleixpol at kde.org
Tue Jun 21 23:11:52 UTC 2016


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/128271/#review96788
-----------------------------------------------------------



LGTM


plugins/mpriscontrol/mpriscontrolplugin.cpp (line 245)
<https://git.reviewboard.kde.org/r/128271/#comment65368>

    Is it really a problem if it's sent even though it's not allowed?


- Aleix Pol Gonzalez


On June 21, 2016, 8:42 p.m., Saikrishna Arcot wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/128271/
> -----------------------------------------------------------
> 
> (Updated June 21, 2016, 8:42 p.m.)
> 
> 
> Review request for kdeconnect.
> 
> 
> Repository: kdeconnect-kde
> 
> 
> Description
> -------
> 
> Send info about whether or not the player can play, pause, go back/forward or seek. The remote side can then disable some actions as necessary.
> 
> Also add a few checks locally to make sure that the command received is valid for this player (as per MPRIS).
> 
> Android side of change is in /r/128273.
> 
> 
> Diffs
> -----
> 
>   plugins/mpriscontrol/mpriscontrolplugin.cpp c83ffb09fa1e36142e83a31cb3bf02f57262b6f4 
> 
> Diff: https://git.reviewboard.kde.org/r/128271/diff/
> 
> 
> Testing
> -------
> 
> Tested with VLC and a client I'm working on that only supports pausing and skipping to the next track.
> 
> 
> Thanks,
> 
> Saikrishna Arcot
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20160621/eee55b9f/attachment.html>


More information about the KDEConnect mailing list