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