search

celluloid / reel

UNMAINTAINED: See celluloid/celluloid#779 - Celluloid::IO-powered web server
https://celluloid.io
MIT License
596 stars 87 forks source link

Patch to solve #87 #88

Closed digitalextremist closed 11 years ago

digitalextremist commented 11 years ago

Solve celluloid/reel#87

coveralls commented 11 years ago

Coverage Status

Coverage decreased (-2.32%) when pulling 9f349f43c909e1ed6ec1002ddf8553e7b57fa817 on digitalextremist:io-work-arou into 470e33de44f8fac410b1ab6c9825ede76e72f42e on celluloid:master.

digitalextremist commented 11 years ago

Actually, there is a better way to do this? ::IO.try_convert()

Please verify this is better and I will update my patch.

/cc: @tarcieri, @halorgium

tarcieri commented 11 years ago

@digitalextremist ::IO.try_convert is unnecessary because @socket will always be a Celluloid::IO socket that responds to #to_io

digitalextremist commented 11 years ago

Roger

tarcieri commented 11 years ago

Eventually I'll add Celluloid::IO.copy_stream to make this a little easier