D17851: Add Android notification backend

Volker Krause noreply at phabricator.kde.org
Sat Jan 12 20:53:04 GMT 2019


vkrause added inline comments.

INLINE COMMENTS

> broulik wrote in notifybyandroid.cpp:79
> Can there be multiple instances of this `NotifyByAndroid`?

KNotificationManager prevents that IIUC.

> broulik wrote in notifybyandroid.cpp:125
> Does this need a `FreeLocalRef` call?

Possible, I'll investigate.

REPOSITORY
  R289 KNotifications

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

To: vkrause, apol
Cc: broulik, apol, nicolasfella, kde-frameworks-devel, michaelh, ngraham, bruns
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20190112/25ae68d2/attachment.html>


More information about the Kde-frameworks-devel mailing list