On Wed, 2 Apr 2025 13:09:01 GMT, Viktor Klang <vkl...@openjdk.org> wrote:

>> test/jdk/java/util/concurrent/tck/ReentrantReadWriteLock20Test.java line 94:
>> 
>>> 92:                         next.join();
>>> 93:                     } catch (InterruptedException ie) {
>>> 94:                     }
>> 
>> I don't think we should swallow InterruptedException here. The only reason 
>> that the main thread will be interrupted here is if jtreg is trying to 
>> timeout the test, so I think it should throw when that happens.
>
> @AlanBateman Yeah, that seems sensible, I'll amend the PR.

Had to force-push since I needed to rebase, but the test should have been 
amended now.

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

PR Review Comment: https://git.openjdk.org/jdk/pull/24261#discussion_r2031296854

Reply via email to