On 04/26/16 23:00, Sebastian Lackner wrote:
The idea of this additional check was to ensure that the request cannot be fulfilled with the current buffer contents. Otherwise its not completely clear that async handler is really waiting for new data, as it should be. If you prefer, I can also remove this test again or set the timeout to 0 to avoid blocking. As the tests show, this part is already working fine. ;)
Yes, please. Both timeout 0 and removing the test is fine with me.
BTW, other patches look good to me.
Thanks, Jacek