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

Ulf Zibis Ulf.Zibis at gmx.de
Wed Aug 17 12:37:23 UTC 2011


As a consequence, allmost all classes in sun.nio.cs and templates in make/tools/CharsetMapping could 
be made package private, at least all access to the mapping tables!

... and don't forget to remove in each sun.nio.cs class and template:

     /**
      * These accessors are temporarily supplied while sun.io
      * converters co-exist with the sun.nio.cs.{ext} charset coders
      * These facilitate sharing of conversion tables between the
      * two co-existing implementations. When sun.io converters
      * are made extinct these will be unncessary and should be removed
      */

     public String getDecoderSingleByteMappings() {
         return b2cTable;
     }

     public short[] getDecoderIndex1() {
         return Decoder.index1;
     }

     public String[] getDecoderIndex2() {
         return Decoder.index2;

     }

     public short[] getEncoderIndex1() {
         return Encoder.index1;

     }
     public String[] getEncoderIndex2() {
         return Encoder.index2;

     }

Additionally, you could mention my wrapper classes [1] in the WORKAROUND section of bug 6237353

[1] http://java.net/projects/java-nio-charset-enhanced/sources/svn/show/trunk/src/sun/io/


-Ulf





More information about the core-libs-dev mailing list