On Thu, 10 Oct 2024 18:01:57 GMT, Phil Race <[email protected]> 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

Reply via email to