D19158: [Runners] Use icon name

Kai Uwe Broulik noreply at phabricator.kde.org
Tue Feb 19 14:25:37 GMT 2019


broulik created this revision.
broulik added a reviewer: KDE Connect.
Herald added a project: KDE Connect.
Herald added a subscriber: kdeconnect.
broulik requested review of this revision.

REVISION SUMMARY
  `QIcon` isn't thread-safe and here we just pass a theme icon name anyway, so we can have the runner UI do the work as needed.

TEST PLAN
  Compiles

REPOSITORY
  R224 KDE Connect

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

AFFECTED FILES
  runners/findmyphone/findmyphonerunner.cpp
  runners/remotecommands/remotecommandsrunner.cpp

To: broulik, #kde_connect
Cc: kdeconnect, ctakano, vporvaznik, mschroeder, varunp, shivanshukantprasad, skymoore, brute4s99, wistak, anoopv, dvalencia, rmenezes, julioc, Leptopoda, timothyc, Danial0_0, johnq, Pitel, adeen-s, SemperPeritus, daniel.z.tg, jeanv, seebauer, bugzy, MayeulC, lemuel, menasshock, mikesomov, tctara, apol
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20190219/43ed112c/attachment.html>


More information about the KDEConnect mailing list