Review Request 127542: Stop clearing the notification list twice
David Kahles
david.kahles96 at gmail.com
Fri Apr 1 13:40:25 UTC 2016
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/127542/
-----------------------------------------------------------
Review request for kdeconnect.
Repository: kdeconnect-kde
Description
-------
There were two clearNotifications() calls when refreshing the notification
list. As there happens nothing relevant in between them, the second call is
useless.
Diffs
-----
interfaces/notificationsmodel.cpp 20d242e3d7b19b0cc6e14f88135f77c5df4018a2
Diff: https://git.reviewboard.kde.org/r/127542/diff/
Testing
-------
None, i don't expect problems with this patch.
Thanks,
David Kahles
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kdeconnect/attachments/20160401/50ca1b61/attachment.html>
More information about the KDEConnect
mailing list