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

Reply via email to