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