[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