Request for review 6237353: Remove sun.io package from j2se binary

[Alan Bateman]

Xueming Shen wrote:
> Hi,
>
> This is something long over due. Some background info.
>
> (1)The java.nio.charset package (to replace the private sun.io 
> package) is added into
>      JDK 1.4 as the result of the nio JSR. Part of the sun.io 
> converters were migrated to
>      the java.nio.charset  implementation in JDK8.
> (2) All supported charsets/converters were migrated to 
> java.nio.charset in JDK5. As the
>      result no more bug fix/rfe maintenance work for sun.io since then.
> (3) Removed all sun.io dependencies (internal sun.io usage inside JDK) 
> from the JDK
>      repository at the beginning of JDK6.
> (4) The sun.io package was removed from the JDK runtime at JDK6 b29.
>
> But we then had one important licensee strongly asked to undo the 
> removal. The sun.io
> was putback in JDK6 6u59/beta.
>
> Now let's try this again in JDK8, the licensee that was against it in 
> JDK6 last time has been
> communicated and is OK with the plan this time. And the JDBC EG group 
> (which heavily
> used sun.io package in the past)  is/will be noticed as well. We 
> assume after 3 major releases
> the risk of breaking someone's application (as the result of this 
> change) is limited.
>
> Here is the webrev. Since we have tried it already (the workspace had 
> been cleaned up
> already) in jdk6 again already, the change this time is relatively 
> simple, mainly only 2 makefile
> changes. The rest is the "deleting" and several minor changes in testing.
>
> http://cr.openjdk.java.net/~sherman/6237353/webrev/
Thanks for doing this, and thanks Lance for checking with all the JDBC 
folks in your phone book.

The changes looks good to me. Push it quick and run :-)

-Alan