Review Request 126471: Don't emit statusChanged if it hasn't changed

David Edmundson david at davidedmundson.co.uk
Tue Dec 22 14:08:29 UTC 2015


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/126471/
-----------------------------------------------------------

Review request for KDE Frameworks and Plasma.


Repository: plasma-framework


Description
-------

For some applets the status is bound to a complex expression that can be
re-evaluated a lot. This ends up causing a lot of unnecassary updates; especially for things in systray.

Found with Aleix's "timertracker" that's in Plasma.


Diffs
-----

  src/plasma/applet.cpp 0b7ef5632c1accb3373aa8f2f57a7880e5f3640d 

Diff: https://git.reviewboard.kde.org/r/126471/diff/


Testing
-------


Thanks,

David Edmundson

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-frameworks-devel/attachments/20151222/568c21a2/attachment.html>


More information about the Kde-frameworks-devel mailing list