[REVIEW] Clock: optimization for font settings

Sebastian Kuegler sebas at kde.org
Wed Apr 16 12:50:34 CEST 2008


On Wednesday 16 April 2008 11:05:47 Rafał Miłecki wrote:
> 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?

please.
-- 
sebas

 http://www.kde.org | http://vizZzion.org |  GPG Key ID: 9119 0EF9 
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 481 bytes
Desc: This is a digitally signed message part.
Url : http://mail.kde.org/pipermail/panel-devel/attachments/20080416/e6561c1a/attachment.pgp 


More information about the Panel-devel mailing list