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

ninghua at linux.vnet.ibm.com ninghua at linux.vnet.ibm.com
Wed Sep 16 05:10:09 UTC 2015


Is there any progress for the patch?


Quoting ninghua at 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





More information about the 2d-dev mailing list