[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-2d-dev
Subject:    [OpenJDK 2D-Dev] Remove redundant package name in src/java.desktop/share/classes/sun/java2d/Disposer
From:       ninghua () linux ! vnet ! ibm ! com
Date:       2015-04-27 2:56:47
Message-ID: 20150426225647.Horde.qBCMviy7AJfKJgmf397ogA1 () ltc ! linux ! ibm ! com
[Download RAW message or body]

This message is in MIME format.


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

["webrev-OJDK-943-OpenJDK9.zip" (application/zip)]

[prev in list] [next in list] [prev in thread] [next in thread]