Bug in onReadable about amp

[bwoebi]

The reason here is that the default chunk size of PHP is 8192. If you read less than that, PHP may have locally buffered data, which just stream_select() is aware of. (Other backends like libevent aren't aware of it either.)
If you use any multiple of 8192 in fread, you won't occur such an issue.
Or you have to explicitly lower the stream chunk size for that stream via stream_set_chunk_size().

It's not per se a bug, but one does have to be aware of that behavior and needs to be documented accordingly.

p.s.: In Amp v2.0 we currently recommend using \AsyncInterop\Loop::onReadable() directly. (but that may change with async-interop/event-loop#149)

[kelunik]

@bwoebi Could you add a PR to async-interop/event-loop regarding that?

[kelunik]

@bwoebi Should probably be stream_set_read_buffer() instead of stream_set_chunk_size()? Or does stream_set_chunk_size() set the size for read and write?