On Sat, 27 Jan 2024 19:43:22 GMT, Jose Pereda <jper...@openjdk.org> wrote:

> However, I'm not sure about the method naming `OnFxThread`, as it might imply 
> that what it does is already done on the FX thread, therefore not needing to 
> be wrapped up by `runOnFxThread`.

I understand what you mean, but the way I see it is that they do already run on 
the FX thread because that's the only reason they exist. I could use something 
like `runMustBeOnFxThread` or `runOnlyFromFxThread` if that's better.
Note that there are already `do___` methods (bad names IMO), and adding 
`run___` might be confusing.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1352#discussion_r1468663135

Reply via email to