D7538: [MPris Engine] Don't construct QDBusObjectPath in Metadata update
Kai Uwe Broulik
noreply at phabricator.kde.org
Fri Aug 25 12:37:36 UTC 2017
broulik created this revision.
Restricted Application added a project: Plasma.
Restricted Application added a subscriber: plasma-devel.
REVISION SUMMARY
Creating one from an empty string prints a warning and here we don't actually need a fully-fledged QDBusObjectPath, we can just compare strings.
BUG: 383557
TEST PLAN
I tested with vlc and plasma-browser-integration and neither of them actually sets mpris:trackid but the warning is gone now
REPOSITORY
R120 Plasma Workspace
REVISION DETAIL
https://phabricator.kde.org/D7538
AFFECTED FILES
dataengines/mpris2/playercontainer.cpp
To: broulik, #plasma
Cc: plasma-devel, ZrenBot, progwolff, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart, lukas
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20170825/a7427850/attachment.html>
More information about the Plasma-devel
mailing list