[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