[patch] KAboutData::programLogo(), QImage used
Jarosław Staniek
js at iidea.pl
Thu Sep 2 14:35:24 BST 2004
> Am Donnerstag, 2. September 2004 13:42 schrieb Jarosław Staniek:
>
>>>On Thu, 2 Sep 2004, [UTF-8] Jarosław Staniek wrote:
>>>
>>>>OK now?
>>>
>>>To save memory and cpu cycles you might want to consider using QImage*
>>>that is initialized to 0 by default. But this consideration depends on
>>>the likelyhood of an logo being set :)
>>
>>Isn't KAboutData object allocated and used mostly once for (typical)
>>application? Otherwise your suggestion could be perfectly right.
>
>
> Yes, but it has been initialized unconditionally whenever the application
> starts. Startup performance shouldn't become worse than it already is, so I
> think Harri's suggestion is very sensible. (I also think that nearly no app
> will change its icon, given the fact that they have been able to do without
> for such a long time.)
that's the point. I will fix this then.
--
regards / pozdrawiam,
Jaroslaw Staniek / OpenOffice Polska
Kexi project: http://koffice.org/kexi/ http://www.kexi-project.org/
Qt-KDE Wrapper project: http://iidea.pl/~js/qkw/
More information about the kde-core-devel
mailing list