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