-
Type:
Bug
-
Resolution: Fixed
-
Priority:
P4
-
Affects Version/s: 24, 25
-
Component/s: client-libs
-
b08
sun.print.PrintJob2D.finalize() calls end()
sun.print.PrintJob2D extends java.awt.PrintJob
java.awt.PrintJob.finalize() also calls end()
So I don't see why sun.print.PrintJob2D needs to over-ride the superclass method.
It can't be over-ridden to be empty because end() still needs to be called until such time as we have a solution for PrintJob.
And whilst removing finalize() on the API class instead is possible from a practical point of view since PrintJob2D is the only concrete class, it would need a spec. update and similarly doesn't actually remove the need for finalization.
sun.print.PrintJob2D extends java.awt.PrintJob
java.awt.PrintJob.finalize() also calls end()
So I don't see why sun.print.PrintJob2D needs to over-ride the superclass method.
It can't be over-ridden to be empty because end() still needs to be called until such time as we have a solution for PrintJob.
And whilst removing finalize() on the API class instead is possible from a practical point of view since PrintJob2D is the only concrete class, it would need a spec. update and similarly doesn't actually remove the need for finalization.
- links to
-
Commit(master)
openjdk/jdk/b8da9695
-
Review(master)
openjdk/jdk/26398