8254798: Deprecate for removal an empty finalize() methods in java.desktop module

Reviewed-by: kcr, pbansal

Author: mrserb

src/java.desktop/share/classes/com/sun/media/sound/AbstractMidiDevice.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1999, 2019, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1999, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -478,12 +478,6 @@ public final MidiDevice getMidiDevice() {
         final boolean isOpen() {
             return open;
         }
-
-        //$$fb is that a good idea?
-        //protected void finalize() {
-        //    close();
-        //}
-
     } // class AbstractReceiver
 
 

src/java.desktop/share/classes/java/awt/color/ICC_Profile.java

@@ -759,7 +759,8 @@ public void activate() throws ProfileDataException {
      *         Object#finalize()} for further information about migration
      *         options.
      */
-    @Deprecated(since="9")
+    @Deprecated(since = "9", forRemoval = true)
+    @SuppressWarnings("removal")
     protected void finalize () {
     }
 

src/java.desktop/share/classes/java/awt/image/ColorModel.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1995, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1995, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -1630,7 +1630,8 @@ public boolean isCompatibleSampleModel(SampleModel sm) {
      *     See the specification for {@link Object#finalize()} for further
      *     information about migration options.
      */
-    @Deprecated(since="9")
+    @Deprecated(since = "9", forRemoval = true)
+    @SuppressWarnings("removal")
     public void finalize() {
     }
 

src/java.desktop/share/classes/java/awt/image/IndexColorModel.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1995, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1995, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -1524,7 +1524,8 @@ public BigInteger getValidPixels() {
      *     See the specification for {@link Object#finalize()} for further
      *     information about migration options.
      */
-    @Deprecated(since="9")
+    @Deprecated(since = "9", forRemoval = true)
+    @SuppressWarnings("removal")
     public void finalize() {
     }
 

src/java.desktop/share/classes/sun/java2d/pipe/RegionClipSpanIterator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1999, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1999, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -377,15 +377,4 @@ public void skipDownTo(int y) {
     public long getNativeIterator() {
         return 0;
     }
-
-    /*
-     * Cleans out all internal data structures.
-     */
-    //public native void dispose();
-
-    @SuppressWarnings("deprecation")
-    protected void finalize() {
-        //dispose();
-    }
-
 }

src/java.desktop/share/classes/sun/java2d/pipe/RegionSpanIterator.java

@@ -1,5 +1,5 @@
 /*
- * Copyright (c) 1999, 2018, Oracle and/or its affiliates. All rights reserved.
+ * Copyright (c) 1999, 2020, Oracle and/or its affiliates. All rights reserved.
  * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
  *
  * This code is free software; you can redistribute it and/or modify it
@@ -47,17 +47,6 @@ public class RegionSpanIterator implements SpanIterator {
     // Is the associated Region rectangular?
     boolean isrect;
 
-/*
-    REMIND: For native implementation
-    long pData;     // Private storage of rect info
-
-    static {
-        initIDs();
-    }
-
-    public static native void initIDs();
-*/
-
     /**
      * Constructs an instance based on the given Region
      */
@@ -197,14 +186,4 @@ public void skipDownTo(int y) {
     public long getNativeIterator() {
         return 0;
     }
-
-    /*
-     * Cleans out all internal data structures.
-     * REMIND: Native implementation
-    public native void dispose();
-
-    protected void finalize() {
-        dispose();
-    }
-     */
 }