On Mon, 24 Mar 2025 17:48:00 GMT, Kevin Rushforth <k...@openjdk.org> wrote:
>> I added a comment explaining where the number comes from. Since it's only >> used in this one spot putting it in a static const variable seemed redundant. >> >> I'm not sure that 250 is a safe limit. I know that the JUnit test crashes if >> this value is set to 252 but succeeds at 251. I don't want to risk >> disturbing existing apps but since each count represents a runnable waiting >> for an event loop to exit even 200 seems like a high number. It might be >> prudent to set this to, say, 246 to give us a bit more headroom. > > I agree that it might be wise to set it a bit lower, so either 246 or even > 240 seems reasonable. Set to 244 so I didn't have to alter the system test. ------------- PR Review Comment: https://git.openjdk.org/jfx/pull/1741#discussion_r2010723341
