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