On Tue, 29 Nov 2022 04:23:56 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> Jaikiran Pai has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Address David's review suggestion > > test/hotspot/jtreg/vmTestbase/nsk/share/locks/DeadlockMaker.java line 31: > >> 29: /* >> 30: * Class used to create deadlocked threads. It is possible to create 2 >> or more deadlocked thread, also >> 31: * it is possible to specify resource of which type should lock each >> deadlocked thread > > Even with the corrections this comment still makes little sense, but lets not > get bogged down trying to improve these old tests. If you feel like it > s/thread,/threads,/ and add a full-stop at the end of the comment. Done. I've updated the PR to include this suggested text. ------------- PR: https://git.openjdk.org/jdk/pull/11386