tarcieri/http — pull request #15 by samphippen (2012-09-05 19:51:26+00:00)
Title: Request#stream refactor

Hello again :)

I had a look around the source base and thought Http::Request#stream could do with a refactor so I broke it out into a [method object](http://c2.com/cgi/wiki?MethodObject) and proceeded to refactor it fairly heavily. I added some specs for some slightly different behaviour.

I think this one might need some more specs before it gets pulled, as I'm relatively certain that the .is_a? behaviour didn't get tested thoroughly enough (look at b4896848587f375f906af9556f765a01e7042a6e and e8d32b8e8d822a52d8bfcdbc355f5aedf5f24793, the specs worked before and after those changes).

All around though I think that this patch significantly improves the readability of that method. Let me know what you think.

Thanks
Sam Phippen