In cases of code like this when closing a stream:
finally {
try {
sampleStream.close();
} catch (IOException e) {
// sorry that this can fail
}
}
I thought it might be a bit cleaner looking to replace the empty try/catch
with Apache Commons IO's IOUtils.closeQuietly(). I noticed that the Apache
Commons IO dependency's scope is currently set to <scope>test</scope>. If
you agree that the code change would be cleaner, is there any problem with
changing that dependency to a compile dependency instead of test?
Thanks,
Jeff