Review Request: New KLauncher DBus interface to check for on hold ioslave for a given url
Dawit Alemayehu
adawit at kde.org
Tue May 10 20:57:16 BST 2011
-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/101333/
-----------------------------------------------------------
Review request for kdelibs.
Summary
-------
As the summary state, this patch adds a new dbus interface to klaucher, checkForHeldSlave, that can be used to check whether or not there is an io slave being held for a given url. This fixes the last known issue with KIO's put-on-hold feature not working properly. With this patch when a user clicks on a PDF link that points to an ftp server, KIO will no longer require multiple connections to service that single request.
Please note that this fix is not specific to the ftp ioslave. Any ioslave that optimizes its copy operation, i.e. supports the canCopyFromFile/canCopyToFile properties, benefits from this fix.
Diffs
-----
kdecore/util/klauncher_iface.h 02c6c9b
kinit/klauncher.h 3fbf2d7
kinit/klauncher.cpp dd45dfe
kinit/klauncher_adaptor.h 0b045c8
kinit/klauncher_adaptor.cpp c7cf086
kio/kio/scheduler.cpp 34f437e
kio/kio/slave.h 780f3c4
kio/kio/slave.cpp 3679ea9
Diff: http://git.reviewboard.kde.org/r/101333/diff
Testing
-------
Thanks,
Dawit
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20110510/2010eb69/attachment.htm>
More information about the kde-core-devel
mailing list