On Wed, 24 Nov 2021 16:53:12 GMT, Kevin Rushforth <k...@openjdk.org> wrote:

>> Jose Pereda has updated the pull request incrementally with one additional 
>> commit since the last revision:
>> 
>>   Address feedback
>
> tests/system/src/test/java/test/com/sun/glass/ui/mac/MacPasteboardTest.java 
> line 56:
> 
>> 54:     public static void setup() throws Exception {
>> 55:         Platform.startup(() -> {
>> 56:             if (PlatformUtil.isMac()) {
> 
> Minor: it might be cleaner to wrap the entire body of the `setup` and 
> `teardown` methods in `if (PlatformUtil.isMac())` so that we don't even start 
> FX on platforms other than macOS.

Done

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

PR: https://git.openjdk.java.net/jfx/pull/668

Reply via email to