RFR: 8297693: Fix typos in src/hotspot and test/hotspot files

David Holmes dholmes at openjdk.org
Tue Nov 29 04:27:59 UTC 2022


On Mon, 28 Nov 2022 09:51:25 GMT, Jaikiran Pai <jpai at openjdk.org> wrote:

> Can I please get a review for this change which only fixes typos in src/hotspot and test/hotspot files? These changes were originally done by @mernst in PR https://github.com/openjdk/jdk/pull/10029, but given that the other PR touches multiple other files and areas, the progress was stalled.
> 
> This PR introduces only hotspot related typo fixes that Michael had proposed in the other PR plus also includes a review suggestion (in one of these files) that was made by Alexey in the other PR.

Looks good and trivial. One further issue spotted around one of the fixes but feel free ignore.

Thanks for taking care of this @jaikiran !

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.

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

Marked as reviewed by dholmes (Reviewer).

PR: https://git.openjdk.org/jdk/pull/11386


More information about the hotspot-dev mailing list