Review Request 122098: Remove redundant calls to lengend->updateContent()

Shalin Shah shalinshah1993 at gmail.com
Tue Jan 20 13:36:25 UTC 2015


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

(Updated Jan. 20, 2015, 1:36 p.m.)


Review request for KDE Edu and Etienne Rebetez.


Changes
-------

Removed comments. 


Repository: kalzium


Description
-------

There is no need to set up slot for updateContent(). Additionally, there is also no need to put updateContent() in slotUpdateSettings() since slotUpdateSettings() already makes a call to slotSwitchtoLookGradient() and slotSwitchtoLookScheme(), both of which update lengends.


Diffs (updated)
-----

  src/kalzium.cpp d23c2aa 

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


Testing
-------


Thanks,

Shalin Shah

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-edu/attachments/20150120/684c35ca/attachment.html>


More information about the kde-edu mailing list