scalaz/scalaz-stream — pull request #66 by tylergreen (2013-12-16 03:25:15+00:00)
Title: changed name of one instance of overloaded method 'linesR' to 'linesStre...

...am'

Running scala 2.10.3, I tried running the example from the readme, substituting 'System.in' for the filename. I would then get the following error:

[error] found : java.io.InputStream

[error] required: String

[error] io.linesR(System.in).

I thought I was doing wrong but after some searching found that method overloading seems to be a discouraged practice in scala, so I changed the name of one of the overloaded methods from 'linesR' to 'linesStream', which solved the problem for me.

Perhaps 'linesStream' isn't the best name but 'linesS' seemed cryptic. In any case, I believe in the future both these methods should be refactored as 3 of their 4 lines are identical. However, I thought I'd introduce the simplest possible fix as I am new to this project and scala in general. Hopefully this makes sense. I look forward to helping improve this awesome library any way I can.