Flush destination IO in IO.copy_stream #4207

Merged
merged 1 commit into from Oct 6, 2016

Projects

None yet

2 participants

@grddev
Contributor
grddev commented Oct 6, 2016

Previously, any buffered IO was ignored when copying to the destination stream.

Untagged a couple of specs that are no longer failing with the additional flush.

Tested this against 9.1.5.0, 9.0.5.0, 1.7.26, and 1.7.19, and buffered IO was ignored in all instances.

@grddev grddev Flush destination IO in IO.copy_stream
Previously, any buffered IO was ignored when copying to the destination stream.

Untagged a couple of specs that are no longer failing with the additional flush.
b7fab70
@headius
Member
headius commented Oct 6, 2016

Very nice, thanks!

@headius headius merged commit 98918e4 into jruby:master Oct 6, 2016

0 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build failed
Details
@headius headius added this to the JRuby 9.1.6.0 milestone Oct 6, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment