KIO scheduler issue

[Waldo Bastian]

Yes, patch is ok, but I don't think it takes care of the actual bug, only the 
symptoms. It very much looks like that for some reason the slave gets killed 
twice.

Cheers,
Waldo

On Friday 19 August 2005 01:43, Paulo Moura Guedes wrote:
> Hi,
>
> The following patch prevents a crash in KLinkStatus, but I don't know if it
> is correct. I would appreciate very much if someone tell me if it is,
> because it would made KLinkStatus a more useful application (#109483,
> #85575).
>
> Index: kdelibs/kio/kio/scheduler.cpp
> ===================================================================
> --- kdelibs/kio/kio/scheduler.cpp       (revision 438634)
> +++ kdelibs/kio/kio/scheduler.cpp       (working copy)
> @@ -626,8 +626,9 @@
>      }
>
>      if (!slaveList->removeRef(slave))
> -       kdDebug(7006) << "Scheduler: BUG!! Slave died, but is NOT in
> slaveList!!!\n" << endl;
> -    slave->deref(); // Delete slave
> +        kdDebug(7006) << "Scheduler: BUG!! Slave " << slave << "/" <<
> slave->slave_pid() << " died, but is NOT in slaveLis$
> +    else
> +        slave->deref(); // Delete slave
>  }
>
>  void Scheduler::slotCleanIdleSlaves()
> ===================================================================
>
> Paulo
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 189 bytes
Desc: not available
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20050819/0748bc57/attachment.sig>