D21779: Resolve X-Flatpak-RenamedFrom

Vlad Zagorodniy noreply at phabricator.kde.org
Thu Jun 13 11:51:21 BST 2019


zzag added inline comments.

INLINE COMMENTS

> notification_p.h:53
>  
> +    static KService::Ptr serviceForDesktopEntry(const QString &desktopEntry);
> +

A bit off topic: is there any point for making this method static? You could get away with making serviceForDesktopEntry function static in the cpp file.

REPOSITORY
  R120 Plasma Workspace

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

To: broulik, #plasma, sitter, davidedmundson
Cc: zzag, plasma-devel, LeGast00n, ericadams, jraleigh, GB_2, ragreen, Pitel, ZrenBot, himcesjf, lesliezhai, ali-mohamed, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20190613/b6a2e09e/attachment.html>


More information about the Plasma-devel mailing list