[icedtea-web][rfc] Move stream closing utility in JNLPClassLoader to StreamUtils

[Omair Majid]

Hi Adam,

On 03/05/2013 03:47 PM, Adam Domurad wrote:
>     * netx/net/sourceforge/jnlp/runtime/JNLPClassLoader: move closeStream
>     to StreamUtils.closeSilently, update usages.
>     * netx/net/sourceforge/jnlp/util/StreamUtils: New method closeSilently,
>     closes stream, catching IOException.

Something like this is better style:

* netx/net/sourceforge/jnlp/runtime/JNLPClassLoader.java
(closeStream): Remove.
(verifySignedJNLP): Make use of StreamUtils instead.
* netx/net/sourceforge/jnlp/utils/StreamUtils.java
(closeSilently): New method.

> +    /***
> +     * Closes a stream, without throwing an exception

Please add a note saying that exceptions will be printed and not just
ignored.

Otherwise looks fine to me.

Cheers,
Omair

-- 
PGP Key: 66484681 (http://pgp.mit.edu/)
Fingerprint = F072 555B 0A17 3957 4E95  0056 F286 F14F 6648 4681