KTar patch to fix speed problems with tar ioslave

David Faure faure at kde.org
Tue Feb 10 13:52:00 CET 2004


On Tuesday 10 February 2004 13:21, Jan Schaefer wrote:
> Hi,
> 
> here is my status report.
> 
> > Well, the copy job doesn't support it yet, so you will need to start there
> > then. The idea is to use a single MultiGetJob instead of multiple Get jobs.
> 
> I looked at the copy job source.
> As far as I understood it, the whole copy job is designed to go recursively 
> through the directories, collect all files and call the the KIO::file_copy 
> method to copy each single file.
> The FileCopyJob handles the get and put calls then.
> The CopyJob itself does not use these low level methods, and as far as I 
> understood the code, it will be much work to use the MultiGet job for copying 
> files. As the code of the CopyJob is really difficult to understand I better 
> do not start to mess around there.

Yes, I think this would be impossible - CopyJob has to take care of "destination
file already exists" and many other cases, which require looking at one file after
another.

I guess your solution is OK since I can't think of another way.

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


More information about the Kde-optimize mailing list