Infinite Loop in KeepAliveStream

[Chris Hegarty]

On 15/12/2012 01:56, Martin Buchholz wrote:
> ...
>
> I was trying to avoid ever calling skip with a larger argument than
> available(), trying to obey the comment
> // Do this ONLY if the skip won't block.
> So my version feels safer, although again I don't have the full context.

OK, makes sense. I'm ok with your version. This code seems to have been 
written with thread-safety "half" in mind, your change is probably the 
safest.

Can you also include the test in the changeset? Or do you want me to 
push it for you?

-Chris.