broulik added a comment.

  +1

INLINE COMMENTS

> notifybyflatpak.h:40
>  public:
> +    struct PortalIcon {
> +        QString str;

Does this need to be in the header file?

REPOSITORY
  R289 KNotifications

BRANCH
  master

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

To: jgrulich, broulik, apol
Cc: apol, kde-frameworks-devel, michaelh, ngraham, bruns

Reply via email to