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. WDYT? --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
