[PATCH/RFC] On the iconview and the speed of directory listing.

David Faure dfaure at klaralvdalens-datakonsult.se
Sun May 25 23:39:31 CEST 2003


On Sunday 25 May 2003 19:32, Maks Orlovich wrote:
> On Sunday 25 May 2003 01:23 pm, David Faure wrote:
> > Very verbose comment - "no iconview" never happens :)
> 
> Then why check for it in KIconViewItem::calcRect()? ;-) 
To prevent a crash, in an impossible situation, like QIconViewItem does ;)

> > Right. Should we sent qt-bugs a patch that delays
> > QIconViewItem::calcRect()? Hmm, sounds a bit difficult, the layouting
> > relies on it.
> >
> > Another idea: we could create items without a parent view (passing 0 to the
> > QIconViewItem constructor), then calling insertItem() to insert it, then
> > calling calcRect() ?
> 
> I am not sure this order will work; QIconViewItem does it the other way around 
> (calcRect() and then insertItem() ); when I tried to reverse it in there as a 
> performance experiment, it blew up rather badly. (KDesktop would run out of 
> memory). 

Oh, then what about calcRect first (but on the KIconViewItem), then insertItem?
Or does your updateItemContainer speedup mean that this isn't necessary anymore?

-- 
David Faure -- faure at kde.org, dfaure at klaralvdalens-datakonsult.se
Qt/KDE/KOffice developer
Klarälvdalens Datakonsult AB, Platform-independent software solutions


More information about the Kde-optimize mailing list