commons-user mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Christian Grobmeier <>
Subject Re: [COMPRESS] change IOUtils.copy() to return number of bytes copied?
Date Sat, 28 Mar 2009 12:49:29 GMT
> IOUtils.copy() currently returns void.
> If it were changed to return a long of the number of bytes actually
> copied, then it could be used in the ArchiveInputStream test cases to
> check that the EOF was reached after the correct number of bytes. E.g.
> that would show whether any padding was being accidentally included.
> The only use of the class in the main code is in the ChangeSet class,
> but of course that does not have to use the return value, and the
> "wasted" code would be minimal.

I think this would make sense and feels more consistent. returns the number of read bytes. If copy() would
do the same, this feels good to me.


To unsubscribe, e-mail:
For additional commands, e-mail:

View raw message