RFR: 8295303: cleanup debug agent's confusing use of EI_GC_FINISH [v2]
Chris Plummer
cjplummer at openjdk.org
Tue Nov 8 02:24:25 UTC 2022
On Tue, 8 Nov 2022 01:16:06 GMT, Alex Menkov <amenkov at openjdk.org> wrote:
>> Chris Plummer has updated the pull request incrementally with one additional commit since the last revision:
>>
>> Fixed ei range check logic errors.
>
> src/jdk.jdwp.agent/share/native/libjdwp/eventHandler.c line 1512:
>
>> 1510: * so it cannot be setup using threadControl_setEventMode(). Use JVMTI API directly.
>> 1511: */
>> 1512: error = JVMTI_FUNC_PTR(gdata->jvmti,SetEventNotificationMode)
>
> Please add space after the comma:
> error = JVMTI_FUNC_PTR(gdata->jvmti, SetEventNotificationMode)
Omitting the space is consistent with other uses of JVMTI_FUNC_PTR.
> src/jdk.jdwp.agent/share/native/libjdwp/util.c line 1983:
>
>> 1981: {
>> 1982: jdwpEvent event = 0;
>> 1983: if (ei >= EI_min || ei >= EI_max) {
>
> Should be "(ei >= EI_min && ei <= EI_max"
Fixed.
> src/jdk.jdwp.agent/share/native/libjdwp/util.c line 1996:
>
>> 1994: {
>> 1995: jvmtiEvent event = 0;
>> 1996: if (ei >= EI_min || ei >= EI_max) {
>
> Should be "(ei >= EI_min && ei <= EI_max"
Fixed.
-------------
PR: https://git.openjdk.org/jdk/pull/10887
More information about the serviceability-dev
mailing list