[ 
https://issues.apache.org/jira/browse/IO-480?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15380652#comment-15380652
 ] 

ASF GitHub Bot commented on IO-480:
-----------------------------------

GitHub user rajivpjs opened a pull request:

    https://github.com/apache/commons-io/pull/14

    [IO-480] Removed the deprectaed method closeQuietly from the Java doc…

    As the closeQuietly method has been deprecated, I thought I would use this 
ticket IO-480 to remove the Java documentation.

You can merge this pull request into a Git repository by running:

    $ git pull https://github.com/rajivpjs/commons-io IO-480

Alternatively you can review and apply these changes as the patch at:

    https://github.com/apache/commons-io/pull/14.patch

To close this pull request, make a commit to your master/trunk branch
with (at least) the following in the commit message:

    This closes #14
    
----
commit 2c801149cbb6fabcef569ac301e9933b169a5fac
Author: Rajiv Jain <[email protected]>
Date:   2016-07-16T09:27:03Z

    [IO-480] Removed the deprectaed method closeQuietly from the Java 
documentation

----


> IOUtils.closeQuietly can throw NullPointerException
> ---------------------------------------------------
>
>                 Key: IO-480
>                 URL: https://issues.apache.org/jira/browse/IO-480
>             Project: Commons IO
>          Issue Type: Bug
>          Components: Utilities
>    Affects Versions: 2.4
>            Reporter: Matt Kusnierz
>   Original Estimate: 10m
>  Remaining Estimate: 10m
>
> The IOUtils#closeQuietly method in its javadoc suggests that ANY exceptions 
> will be ignored. This is not the case, as the code only catches IOExceptions. 
> As this example stack trace shows, it is possible for some InputStreams to 
> throw NullPointerExceptions during close:
> java.lang.NullPointerException: null
>       at java.io.FilterInputStream.close(FilterInputStream.java:155) 
> ~[na:1.6.0_35]
>       at 
> sun.net.www.protocol.jar.JarURLConnection$JarURLInputStream.close(JarURLConnection.java:90)
>  ~[na:1.6.0_35]
>       at org.apache.commons.io.IOUtils.closeQuietly(IOUtils.java:281) 
> ~[commons-io-2.1.jar:2.1]
>       at org.apache.commons.io.IOUtils.closeQuietly(IOUtils.java:224) 
> ~[commons-io-2.1.jar:2.1]
> Close quietly should do as advertised/suggested and catch all Exceptions.



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to