<table><tr><td style="">marten added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D28647">View Revision</a></tr></table><br /><div><div><p>I'm happy to work on the refactoring if you think it's the right thing to do.</p>

<p>Do you mean splitting SchedulerPrivate::slotReparseSlaveConfiguration() up into two halves, the first part (KProtocolManager::reparseConfiguration through to NetRC::self()->reload - reparsing the configuration in the current process) being called directly and by the DBus signal, while the second half (check that 'proto' is applicable then iterate through the allSlaves() list) being called only from emitReparseSlaveConfiguration?  Something like:</p>

<div class="remarkup-code-block" style="margin: 12px 0;" data-code-lang="text" data-sigil="remarkup-code-block"><pre class="remarkup-code" style="font: 11px/15px "Menlo", "Consolas", "Monaco", monospace; padding: 12px; margin: 0; background: rgba(71, 87, 120, 0.08);">void Scheduler::emitReparseSlaveConfiguration()
{
  schedulerPrivate()->slotReparseSlaveConfiguration(...);
  schedulerPrivate()->reparseOtherSlaves();
}

void SchedulerPrivate::slotReparseSlaveConfiguration(...)
{
  KProtocolManager::reparseConfiguration();
  ,,,
  NetRC::self()->reload();
}

void SchedulerPrivate::reparseOtherSlaves()
{
  check protocol, return if not applicable
  iterate over allSlaves()
  {
    slave->send(CMD_REPARSECONFIGURATION); slave->resetHost();
  }
}</pre></div></div></div><br /><div><strong>REPOSITORY</strong><div><div>R241 KIO</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D28647">https://phabricator.kde.org/D28647</a></div></div><br /><div><strong>To: </strong>marten, Frameworks, dfaure<br /><strong>Cc: </strong>kde-frameworks-devel, LeGast00n, cblack, michaelh, ngraham, bruns<br /></div>