----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: http://git.reviewboard.kde.org/r/104120/#review11142 -----------------------------------------------------------
This review has been submitted with commit 7a5ecbd5316b880f86a2088f420057cdb8d4e2ba by Dawit Alemayehu to branch KDE/4.8. - Commit Hook On March 5, 2012, 3:46 p.m., Dawit Alemayehu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > http://git.reviewboard.kde.org/r/104120/ > ----------------------------------------------------------- > > (Updated March 5, 2012, 3:46 p.m.) > > > Review request for kdelibs and Andreas Hartmetz. > > > Description > ------- > > This patch is an alternate approach to making the number of ioslaves > connections allowed per server configurable by the user. Unlike the patch > proposed in bug# 64647, the approach taken in this patch neither requires any > new API additions to KIO::Scheduler nor does it limit configurablity per > protocol. > > For restricting the number of ioslaves per server/host all a user would have > to do is add a "MaxInstancesPerHost" property to the specific protocols > configuration file. For example, to limit the number of ftp connections to > "ftp://ftp.foo.bar" to 1, one would only have to add the following to > $KDEHOME/share/config/kio_ftprc: > > [ftp.foo.bar] > MaxInstancesPerHost=1 > > But that is not all. The user can make that restriction apply globally to all > ftp connections by simply moving property to the global section of of the > configuration file. That is move it out of the host ("[ftp.foo.bar]") > section. And to make it apply to every protocol, simply move the property to > the generic ioslave configuration file, $KDEHOME/share/config/kioslaverc. > > > This addresses bug 64647. > http://bugs.kde.org/show_bug.cgi?id=64647 > > > Diffs > ----- > > kio/kio/scheduler.cpp 6b4ed52 > > Diff: http://git.reviewboard.kde.org/r/104120/diff/ > > > Testing > ------- > > To verify correct behavior, I added the following settings to kio_ftprc and > attempted to download multiple files at the same time. KIO::Scheduler > correctly processed the request one at a time: > > [ftp.kde.org] > MaxConnections=1 > > > Thanks, > > Dawit Alemayehu > >