-
Bug
-
Resolution: Fixed
-
P4
-
24, 25
-
master
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