RFR: 8357157: Exception thrown from AnimationTimer freezes application [v2]

Michael Strauß mstrauss at openjdk.org
Fri May 16 18:40:54 UTC 2025


On Fri, 16 May 2025 18:27:14 GMT, Andy Goryachev <angorya at openjdk.org> wrote:

>> Michael Strauß has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   review comments
>
> modules/javafx.graphics/src/main/java/com/sun/scenario/animation/AbstractPrimaryTimer.java line 380:
> 
>> 378:                     Thread thread = Thread.currentThread();
>> 379:                     thread.getUncaughtExceptionHandler().uncaughtException(thread, e);
>> 380:                 } catch (Throwable ignored) {
> 
> any reason not to wrap the whole method in `try`/`catch`?

There isn't anything else in there that could throw, except for the logger itself. I don't think it makes sense to try to account for failing loggers, as basically everything will break if that's the case.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1811#discussion_r2093520616


More information about the openjfx-dev mailing list