Code review request: 7068616 NIO libraries do not build with javac -Xlint:all,-deprecation -Werror

[Chris Hegarty]

Hi Sasha,

Thanks for cleaning up these warnings.

I reviewed all the changes, paying particular attention to SCTP, and I'm 
happy with them.

One minor comment, for Charset.java

   Since providers() now returns a Iterator<CharsetProvider>,
   the callers of this method could also be updated to use the
   generified type and remove (what would be) a redundant cast.

Thanks,
-Chris.

On 07/27/11 10:42 PM, Alexandre Boulgakov wrote:
>   Hello everyone,
>
> Please review these NIO changes.
> Bug detail: http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7068616
> <http://bugs.sun.com/bugdatabase/view_bug.do?bug_id=7068616>
> webrev: http://cr.openjdk.java.net/~alanb/7068616/webrev/
>
> Most of the changes are one-line fixes to remove warnings, and I've
> updated the make files to flag warnings as errors from now on, ignoring
> deprecation warnings in sun.io.*. I also revised some of the try-catch
> blocks to make use of multicatch syntax.
>
> There are two larger changes in
> src/solaris/classes/sun/nio/ch/SctpChannelImpl.java and
> src/solaris/classes/sun/nio/ch/SctpMultiChannelImpl.java, where I
> changed a private inner class from "private class
> InternalNotificationHandler<T> extends AbstractNotificationHandler<T>"
> to "private class InternalNotificationHandler extends
> AbstractNotificationHandler<Object>" because it was not using the type
> variable T. AbstractNotificationHandler<Object> is consistent with the
> original behavior where InternalNotificationHandler was used as a raw type.
>
> Thanks,
> Sasha