[PATCH] ported kurlcompletion.cpp!DirectoryListThread::run() to use Qt APIs

[David Faure]

On Friday 02 May 2008, Michael O'Shea wrote:
> If I'm right, the only thing that will vary is how quickly the dropdown list
> in Dolphin (in the case I'm interested in) fills up.

Well, that and other uses of completion, yes. The speed with which the GUI reacts
is exactly what users perceive as "performance", so it shouldn't be made slow :)

> currentDir.entryInfoList() seems costly for large directories, maybe
> > QDirIterator is better?
> >
> 
> I don't mind giving it a try but would you agree that the asynchronous mode
> of operation in kurlcompletion.cpp makes the distinction not so important ?

Well, entryInfoList also allocates much more memory... an extremely large directory
could make your machine swap -- that's unrelated to sync vs async...

-- 
David Faure, faure at kde.org, sponsored by Trolltech to work on KDE,
Konqueror (http://www.konqueror.org), and KOffice (http://www.koffice.org).