[REVIEW] Clock: optimization for font settings

Rafał Miłecki zajec5 at gmail.com
Wed Apr 16 11:05:47 CEST 2008


In KConfigGroup we store full info (family, bold, italic) about font
using QFont. Then there is no reason for using additional variables
for storing info about font's status of bold and status of italic.

Moreover we don't need copy of m_plainClockFont inside
Clock::preparePainter. I don't see a reason why we should operate on
copy of original font.

Can commit?

-- 
Rafał Miłecki
-------------- next part --------------
A non-text attachment was scrubbed...
Name: clock.font.settings.optimization.patch
Type: application/octet-stream
Size: 4153 bytes
Desc: not available
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080416/5e7019b3/attachment.obj 


More information about the Panel-devel mailing list