<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/100819/">http://git.reviewboard.kde.org/r/100819/</a>
</td>
</tr>
</table>
<br />
<table bgcolor="#fefadf" width="100%" cellspacing="0" cellpadding="8" style="background-image: url('http://git.reviewboard.kde.orgrb/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.</div>
<div>By Dawit Alemayehu.</div>
<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;">The dbus service check in KIO::SchedulerPrivate::slotReparseSlaveConfiguration always return true so long as the reparse request originats from within the application, e.g. a config dialog. That unfortunately prevents KIO::Scheduler from updating already running instances of ioslaves and as such changes won't take effect until you close the application and restart it. For example, if you change the proxy settings in Konqueror through its config dialog, those changes will not take effect until you restart Konqueror.
This patch addresses this issue of config changes not taking effect immediately without breaking what seems to be the intention the problematic check mentioned above ; ensuring slotReparseSlaveConfiguration is not called twice whenever the public static function KIO::Scheduler::emitReparseSlaveConfiguration function is invoked.
</pre>
</td>
</tr>
</table>
<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">(d5fab85)</span></li>
</ul>
<p><a href="http://git.reviewboard.kde.org/r/100819/diff/" style="margin-left: 3em;">View Diff</a></p>
</td>
</tr>
</table>
</div>
</body>
</html>