Review Request 119211: Queue CMD_REPARSECONFIGRATION requests for suspended ioslaves

David Faure faure at kde.org
Thu Jul 10 18:08:13 BST 2014


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://git.reviewboard.kde.org/r/119211/#review62085
-----------------------------------------------------------


But why CMD_REPARSECONFIGURATION especially?

If the slave is on hold, will anything else get through? IOW should the test just be extended with "|| suspended", no test on cmd?

- David Faure


On July 10, 2014, 12:04 p.m., Dawit Alemayehu wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://git.reviewboard.kde.org/r/119211/
> -----------------------------------------------------------
> 
> (Updated July 10, 2014, 12:04 p.m.)
> 
> 
> Review request for kdelibs, David Faure and Thiago Macieira.
> 
> 
> Repository: kdelibs
> 
> 
> Description
> -------
> 
> This patch addresses the issue where attempting to send CMD_REPARSECONFIGURATION to suspended ioslaves results in a freezing of the application that sent the request. This problem can be clearly reproduced by following the steps outlined in the bug report.
> 
> I am not entirely sure whether or not this fix really belongs in kio/connection.cpp, but I could not find a more convenient location to queue the command request to ensure the ioslave receives the reparse configuration command if it is ever reused again.
> 
> 
> Diffs
> -----
> 
>   kio/kio/connection.cpp 99aea0b 
> 
> Diff: https://git.reviewboard.kde.org/r/119211/diff/
> 
> 
> Testing
> -------
> 
> Run the steps given in the bug report before and after the fix.
> 
> 
> Thanks,
> 
> Dawit Alemayehu
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-core-devel/attachments/20140710/76de0a6a/attachment.htm>


More information about the kde-core-devel mailing list