On Tue, 14 Nov 2023 06:50:58 GMT, Roman Marchenko <rmarche...@openjdk.org> 
wrote:

> Zero'ing memory of extension event callbacks

Marked as reviewed by dholmes (Reviewer).

src/hotspot/share/prims/jvmtiEnvBase.cpp line 217:

> 215:   // all callbacks initially null
> 216:   memset(&_event_callbacks,0,sizeof(jvmtiEventCallbacks));
> 217:   memset(&_ext_event_callbacks, 0, sizeof(jvmtiExtEventCallbacks));

Looks good.

While you are here could you adjust the existing memset line and add spaces 
after the commas please. Thanks.

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

PR Review: https://git.openjdk.org/jdk/pull/16647#pullrequestreview-1729084003
PR Review Comment: https://git.openjdk.org/jdk/pull/16647#discussion_r1392118874

Reply via email to