[prev in list] [next in list] [prev in thread] [next in thread] 

List:       kde-core-devel
Subject:    Re: PATCH: animated widget resizing
From:       Dirk Mueller <mueller () kde ! org>
Date:       2000-08-29 17:14:38
[Download RAW message or body]

On Mon, 28 Aug 2000, Kurt Granroth wrote:

> 
> The reason it doesn't change correctly is because Konqueror passes
> along a list of icons to use that are all *absolute* pathnames.  The
> absolute pathname prevents KIconLoader from loading the correct size.

Okay. Just make sure that the iconloader doesn't stat/access() each
frame of the animation separately, because that's terrible slow. 

In case of the konqueror animated widget this is an overhead of
1000 stat calls, 1000 access calls and a lot of other stuff summarizing
to about 0.14 seconds on my machine. 

That's why I changed it to save a list of the icons. 

Dirk

[prev in list] [next in list] [prev in thread] [next in thread]