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

Reply via email to