Hi,
I was replacing my own copy of CharSequenceReader with the one from
commons-io, but I had some test failures. I found out that there were
two reasons:
* It doesn't override ready(), and therefore always returns false, even
if there are still characters to be read. That's not incorrect according
to the specification, but it's better to return true instead.
Pull request: https://github.com/apache/commons-io/pull/122
* skip(long) currently returns -1 if the stream has ended. However, that
violates the contract of Reader.skip(long), which says the method should
return the number of characters actually skipped. At the end of the
stream, that's not -1 but 0.
Pull request: https://github.com/apache/commons-io/pull/123
Kind regards,
Rob
---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscr...@commons.apache.org
For additional commands, e-mail: dev-h...@commons.apache.org
