On Tue, 8 Nov 2022 19:27:18 GMT, Chris Plummer <cjplum...@openjdk.org> wrote:

>> src/jdk.jdwp.agent/share/native/libjdwp/error_messages.c line 236:
>> 
>>> 234:         CASE_RETURN_TEXT(EI_CLASS_PREPARE)
>>> 235:         CASE_RETURN_TEXT(EI_CLASS_UNLOAD)
>>> 236:         CASE_RETURN_TEXT(EI_CLASS_LOAD)
>> 
>> Nit: Would it be a little better to place it after EI_CLASS_UNLOAD?
>
> If you mean put EI_CLASS_UNLOAD after EI_CLASS_LOAD, I tried to keep the 
> current ordering. If you look in util.h (bottom of this page), it used to 
> have:
> 
>         EI_CLASS_PREPARE        =  7,
>         EI_GC_FINISH            =  8,
>         EI_CLASS_LOAD           =  9,
> 
> and now it has:
> ``` 
>         EI_CLASS_PREPARE        =  7,
>         EI_CLASS_UNLOAD         =  8,
>         EI_CLASS_LOAD           =  9,
> 
> There are a couple of other places where this ordering is preserved and I 
> tried to be consistent.

Okay, thanks.
It makes sense then.

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

PR: https://git.openjdk.org/jdk/pull/10887

Reply via email to