Re: [OpenJDK 2D-Dev] Remove redundant package name in src/java.desktop/share/classes/sun/java2d/Disposer.java

[ninghua]

Is there any progress for the patch?


Quoting ning...@linux.vnet.ibm.com:

A. The problem to be resolve or feature to be added

The class name includes package name java.security/java.lang.ref though they were already imported

B. The solution proposed by this patch
Remove package name java.security/java.lang.ref from class name

C: Modified file
src/java.desktop/share/classes/sun/java2d/Disposer.java

D:Patch

--- old/src/java.desktop/share/classes/sun/java2d/Disposer.java 2014-09-02 10:04:34.098277541 +0800 +++ new/src/java.desktop/share/classes/sun/java2d/Disposer.java 2014-09-02 10:04:33.918277545 +0800

@@ -61,15 +61,15 @@
     public static int refType = PHANTOM;

     static {
-        java.security.AccessController.doPrivileged(
-            new java.security.PrivilegedAction<Void>() {
+        AccessController.doPrivileged(
+            new PrivilegedAction<Void>() {
                 public Void run() {
                     System.loadLibrary("awt");
                     return null;
                 }
             });
         initIDs();
-        String type = java.security.AccessController.doPrivileged(
+        String type = AccessController.doPrivileged(

new sun.security.action.GetPropertyAction("sun.java2d.reftype"));

         if (type != null) {
             if (type.equals("weak")) {
@@ -134,7 +134,7 @@
         if (target instanceof DisposerTarget) {
             target = ((DisposerTarget)target).getDisposerReferent();
         }
-        java.lang.ref.Reference<Object> ref;
+        Reference<Object> ref;
         if (refType == PHANTOM) {
             ref = new PhantomReference<>(target, queue);
         } else {

It is attached as webrev-OJDK-943-OpenJDK9.zip


Regards,

NingHua