On Mon, 2 Nov 2020 16:22:57 GMT, Coleen Phillimore <cole...@openjdk.org> wrote:

>> src/hotspot/share/prims/jvmtiTagMap.cpp line 345:
>> 
>>> 343: 
>>> 344:   // Check if we have to process for concurrent GCs.
>>> 345:   check_hashmap(false);
>> 
>> Maybe add a comment stating the parameter name, as was done in other 
>> callsites for check_hashmap.
>
> Ok, will I run afoul of the ZGC people putting the parameter name after the 
> parameter and the rest of the code, it is before?

ZGC people passionately place the comment after the argument value.

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

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

Reply via email to