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