RFR: 8341257: Open source few DND tests - Set1 [v6]

Damon Nguyen dnguyen at openjdk.org
Thu Oct 10 18:52:13 UTC 2024


On Thu, 10 Oct 2024 18:01:57 GMT, Phil Race <prr at openjdk.org> wrote:

>> Damon Nguyen has updated the pull request incrementally with one additional commit since the last revision:
>> 
>>   Add dispose
>
> test/jdk/java/awt/dnd/DnDClipboardDeadlockTest.java line 345:
> 
>> 343: 
>> 344:     public void dragDropEnd(DragSourceDropEvent dsde) {
>> 345:         System.exit(DnDClipboardDeadlockTest.CODE_OK);
> 
> SFAICT you haven't done anything to ensure this is ONLY the child process calling System.exit()

Do you mean via a lock or am I misunderstanding this? I assumed this `exit()` was called once because there is only one instance of DragSourceButton/DragSourcePanel which would be the only one calling `exit()`.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/21394#discussion_r1795937816


More information about the client-libs-dev mailing list