[jdk8u-ri] RFR: 8287132: Retire Runtime.runFinalizersOnExit so that it always throws UOE [v4]

[Bernd Eckenfels]

Hello,

Can I ask a stupid question? How is this planned to float back to the updates project and the openjdk8 distributions which are out there?

 I could imagine this has the potential to break existing programs if symbols get removed or if a function throws an exception. I would not expect such a removal in a version which is in maintenance. What am I understand here wrong? Why such maintenance releases to the 8u version anyway?

Gruss
Bernd


--
http://bernd.eckenfels.net
________________________________
Von: jdk8u-dev <jdk8u-dev-retn at openjdk.java.net> im Auftrag von Iris Clark <iris at openjdk.java.net>
Gesendet: Friday, May 27, 2022 7:58:44 AM
An: jdk8u-dev at openjdk.java.net <jdk8u-dev at openjdk.java.net>
Betreff: Re: [jdk8u-ri] RFR: 8287132: Retire Runtime.runFinalizersOnExit so that it always throws UOE [v4]

On Wed, 25 May 2022 01:26:56 GMT, David Holmes <dholmes at openjdk.org> wrote:

>> Please review these further changes for JDK 8 Maintenance Release 4 - JSR-337.
>>
>> The MR4 changes to the reference processing implementation have made continued support for `runFinalizersonExit` unreasonable and so it is "retired" and will always throw `UnsupportedOperationException`. (This is what was also done for `Thread.stop(Throwable)` in JDK 8.)
>>
>> Various updates to the specifications are made to remove references to running finalizers at exit - see the CSR request for details.
>>
>> All dead code (including the already dead `JVM_Exit`) is removed. Shutdown.java is replaced by the version from mainline with a slight modification as 8u doesn't have `sun.misc.VM.isShutdown()`.
>>
>> Tests using RFOE are deleted and a new test to check the throwing of UOE added. The new test was NOT a rename of an existing test but for some reason git is treating it that way.
>>
>> Testing: :jdk_core group
>>
>> Thanks,
>> David
>
> David Holmes has updated the pull request incrementally with one additional commit since the last revision:
>
>   Fix style nit

Spec changes in java.lang.{Runtime,Shutdown,System} align with CSR 8287133 and are as expected.

-------------

Marked as reviewed by iris (Author).

PR: https://git.openjdk.java.net/jdk8u-ri/pull/9