RFR: 8318841: macOS: Memory leak with MenuItem when Menu.useSystemMenuBar(true) is used [v4]

Johan Vos jvos at openjdk.org
Thu Nov 9 09:16:10 UTC 2023


On Wed, 8 Nov 2023 18:26:14 GMT, Kevin Rushforth <kcr at openjdk.org> wrote:

>> Johan Vos has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add a system test
>
> tests/system/src/test/java/test/javafx/stage/SystemMenuBarTest.java line 122:
> 
>> 120:             Thread.currentThread().setUncaughtExceptionHandler((t,e) -> {
>> 121:                 e.printStackTrace();
>> 122:                 failed.set(true);
> 
> To avoid the test hanging indefinitely, I recommend adding the following after setting the `failed` flag:
> 
> 
>     memoryLatch.countDown();

good idea.

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

PR Review Comment: https://git.openjdk.org/jfx/pull/1277#discussion_r1387700709


More information about the openjfx-dev mailing list