On Mon, 8 May 2023 05:25:04 GMT, David Holmes <dhol...@openjdk.org> wrote:
>> src/hotspot/share/utilities/resourceHash.hpp line 147:
>>
>>> 145: */
>>> 146: bool put_fast(K const& key, V const& value) {
>>> 147: unsigned hv = HASH(key);
>>
>> I think `put_fast` is not clear enough. Maybe `put_must_be_absent()` or
>> something more concise.
>
> I would suggest `put_when_absent` to complement `put_if_absent` - with
> suitable descriptive comments of course.
This is a good name. Updated.
-------------
PR Review Comment: https://git.openjdk.org/jdk/pull/13818#discussion_r1187483386
