<html>
<body>
<div style="font-family: Verdana, Arial, Helvetica, Sans-Serif;">
<table bgcolor="#f9f3c9" width="100%" cellpadding="8" style="border: 1px #c9c399 solid;">
<tr>
<td>
This is an automatically generated e-mail. To reply, visit:
<a href="http://git.reviewboard.kde.org/r/104120/">http://git.reviewboard.kde.org/r/104120/</a>
</td>
</tr>
</table>
<br />
<p>Ship it!</p>
<pre style="white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Looks good to me, thanks for this long-due fix.
I don't know how well the underlying feature works (e.g. will the jobs wait for a connection or fail immediately, and will getConnectedSlave return 0 when the limit is reached) -- but this patch looks ok, it's just "more (configurability) of the same (feature)".</pre>
<br />
<div>
<table width="100%" border="0" bgcolor="white" style="border: 1px solid #C0C0C0; border-collapse: collapse; margin: 2px padding: 2px;">
<thead>
<tr>
<th colspan="4" bgcolor="#F0F0F0" style="border-bottom: 1px solid #C0C0C0; font-size: 9pt; padding: 4px 8px; text-align: left;">
<a href="http://git.reviewboard.kde.org/r/104120/diff/2/?file=51686#file51686line737" style="color: black; font-weight: bold; text-decoration: underline;">kio/kio/scheduler.cpp</a>
<span style="font-weight: normal;">
(Diff revision 2)
</span>
</th>
</tr>
</thead>
<tbody style="background-color: #e4d9cb; padding: 4px 8px; text-align: center;">
<tr>
<td colspan="4"><pre style="font-size: 8pt; line-height: 140%; margin: 0; ">public:</pre></td>
</tr>
</tbody>
<tbody>
<tr>
<th bgcolor="#b1ebb0" style="border-right: 1px solid #C0C0C0;" align="right"><font size="2"></font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "></pre></td>
<th bgcolor="#b1ebb0" style="border-left: 1px solid #C0C0C0; border-right: 1px solid #C0C0C0;" align="right"><font size="2">737</font></th>
<td bgcolor="#c5ffc4" width="50%"><pre style="font-size: 8pt; line-height: 140%; margin: 0; "> <span class="c1">// Never allow maxSlavesPerHost to exceed maxSalves.</span></pre></td>
</tr>
</tbody>
</table>
<pre style="margin-left: 2em; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">Typo (salves)</pre>
</div>
<br />
<p>- David</p>
<br />
<p>On March 3rd, 2012, 5:48 p.m., Dawit Alemayehu wrote:</p>
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.org/media/rb/images/review_request_box_top_bg.png'); background-position: left top; background-repeat: repeat-x; border: 1px black solid;">
<tr>
<td>
<div>Review request for kdelibs and Andreas Hartmetz.</div>
<div>By Dawit Alemayehu.</div>
<p style="color: grey;"><i>Updated March 3, 2012, 5:48 p.m.</i></p>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Description </h1>
<table width="100%" bgcolor="#ffffff" cellspacing="0" cellpadding="10" style="border: 1px solid #b8b5a0">
<tr>
<td>
<pre style="margin: 0; padding: 0; white-space: pre-wrap; white-space: -moz-pre-wrap; white-space: -pre-wrap; white-space: -o-pre-wrap; word-wrap: break-word;">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.</pre>
</td>
</tr>
</table>
<div style="margin-top: 1.5em;">
<b style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Bugs: </b>
<a href="http://bugs.kde.org/show_bug.cgi?id=64647">64647</a>
</div>
<h1 style="color: #575012; font-size: 10pt; margin-top: 1.5em;">Diffs</b> </h1>
<ul style="margin-left: 3em; padding-left: 0;">
<li>kio/kio/scheduler.cpp <span style="color: grey">(6b4ed52)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/104120/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>