On Fri, 9 Apr 2021 16:15:28 GMT, Daniel D. Daugherty <[email protected]> wrote:

>> src/hotspot/share/runtime/handshake.cpp line 632:
>> 
>>> 630: }
>>> 631: 
>>> 632: void HandshakeState::self_suspened() {
>> 
>> Typo: s/self_suspened/self_suspended/
>
> I'm not fond of `self_suspended` as the function name.
> Perhaps `do_self_suspend` or something like that...
> (What color would you like that bike shed?)

I like the suggestion to rename self_suspened to do_self_suspend.

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

PR: https://git.openjdk.java.net/jdk/pull/3191

Reply via email to