[
https://issues.apache.org/jira/browse/IO-504?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15274941#comment-15274941
]
Gary Gregory commented on IO-504:
---------------------------------
Thanks for the updated patch.
I think it is important to note that there is an important semantic change here.
With the patch, whenever [io] internally tries to close a closeable something
(file, stream, and so on), we now may get an exception when before we did not.
I'd argue that ignoring all of these close exception was bad and could cause
things to get polluted or corrupted or the call sites to be unaware that
problems exists closing resources. Not a good thing especially when other
IOExceptions would be thrown on other operations within a given method.
> Deprecated of all IOUtils.closeQuietly() methods
> ------------------------------------------------
>
> Key: IO-504
> URL: https://issues.apache.org/jira/browse/IO-504
> Project: Commons IO
> Issue Type: Wish
> Reporter: Christian Schulte
> Attachments: IO-504.patch, IO-504.patch
>
>
> As soon as 'commons-io' is targetted at Java 7, all 'IOUtils.closeQuietly'
> methods should be deprecated and people should be told to use the
> try-with-resources statement. Those methods are way to error prone and used
> incorrectly almost everywhere. If 'commons-io' has '-target 1.7', keeping
> those methods makes no sense anymore.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
