RFR: 8362291: [macOS] Remove finalize method in CGraphicsEnvironment.java [v2]

Sergey Bylokhov serb at openjdk.org
Wed Jul 16 22:07:00 UTC 2025


On Wed, 16 Jul 2025 20:57:40 GMT, Phil Race <prr at openjdk.org> wrote:

>> Remove a finalize() method in CGraphicsEnvironment, replacing it with Disposer.
>> 
>> I don't see a way to add a test to verify this clean up.
>
> Phil Race has updated the pull request incrementally with one additional commit since the last revision:
> 
>   8362291

src/java.desktop/macosx/classes/sun/awt/CGraphicsEnvironment.java line 157:

> 155:                 deregisterDisplayReconfiguration(displayReconfigContext);
> 156:             } catch (Throwable t) {
> 157:                 // Runs on disposer thread, can't allow an exception to escape.

What do you think about moving this "catch Throwable" into Disposer itself? As of now since https://github.com/openjdk/jdk/pull/9890 we ignore "Exception" only.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/26332#discussion_r2211699632


More information about the client-libs-dev mailing list