RFR: JDK-8297414: Remove easy warnings in javafx.controls [v3]

John Hendrikx jhendrikx at openjdk.org
Wed Nov 23 23:12:36 UTC 2022


On Wed, 23 Nov 2022 22:57:07 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> This looks like bad test design. If the toolkit needs to be of a specific type when running tests, then check that this is the case once before running all the tests. As for this PR, I don't mind leaving it as is or reverting these changes. I don't think that a CCE is realistic here.
>
>> As for this PR, I don't mind leaving it as is or reverting these changes. I don't think that a CCE is realistic here.
> 
> I agree that this isn't a good test design. I was just pointing out why it isn't (in theory) a completely compatible change. I don't really mind one way or the other here.

I can replace these with an assert:

    assertTrue(Toolkit.getToolkit() instanceof StubToolkit);

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

PR: https://git.openjdk.org/jfx/pull/959


More information about the openjfx-dev mailing list