RFR: 8351733: Crash when creating too many nested event loops [v5]

Martin Fox mfox at openjdk.org
Thu Mar 27 19:42:21 UTC 2025


On Thu, 27 Mar 2025 17:33:23 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Martin Fox has updated the pull request incrementally with two additional commits since the last revision:
>> 
>>  - Removed unnecessary import
>>  - The max nested event loop constant is no longer public
>
> modules/javafx.graphics/src/main/java/javafx/application/Platform.java line 305:
> 
>> 303:      * other than the JavaFX Application Thread.
>> 304:      *
>> 305:      * @throws RuntimeException if this call would exceed the maximum
> 
> I wonder if there is a more specific subclass of `RuntimeException` that we could throw? Maybe `IllegalStateException`, which seems the closest in meaning and has the advantage that we already throw it in other cases.

I'll have to wait for others to chime in on that question, I don't have much background in this area.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1741#discussion_r2017499898


More information about the openjfx-dev mailing list