On Mon, 2 Jun 2025 16:52:26 GMT, Alisen Chung <ach...@openjdk.org> wrote:
>> Matthias Bläsing has updated the pull request incrementally with one
>> additional commit since the last revision:
>>
>> Add headful key to the jtreg description
>
> test/jdk/java/awt/Clipboard/ConcurrentClipboardAccessTest.java line 31:
>
>> 29: @run main ConcurrentClipboardAccessTest
>> 30: */
>> 31: import java.awt.*;
>
> expand imports
Done
> test/jdk/java/awt/Clipboard/ConcurrentClipboardAccessTest.java line 58:
>
>> 56: public void run() {
>> 57: final Clipboard systemClipboard =
>> Toolkit.getDefaultToolkit().getSystemClipboard();
>> 58: while (true) {
>
> will this cause the test to run until timeout if the thread isn't interrupted?
Valid point. Adjusted test to use daemon threads for the concurrent access
tests. I reran the tests on the baseline JDK and after the changes to verify
the findings from
https://github.com/openjdk/jdk/pull/24614#issuecomment-2884495131. Did not
notice this as the jtreg harness indeed seems to terminate the JVM and not
report that as a failure, tested now also manually (without jtreg).
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/24614#discussion_r2124872594
PR Review Comment: https://git.openjdk.org/jdk/pull/24614#discussion_r2124872455
