On Thu, 4 Jun 2026 18:24:09 GMT, Artur Barashev <[email protected]> wrote:
>> Sean Mullan has updated the pull request incrementally with one additional >> commit since the last revision: >> >> Switch order of fields. > > src/java.base/share/classes/jdk/internal/event/TLSHandshakeEvent.java line 38: > >> 36: public String cipherSuite; >> 37: public long certificateId; >> 38: public String namedGroup; > > Nit: `namedGroup` is being logged right after `cipherSuite`, it will be > consistent to keep the same order here and in > [`src/jdk.jfr/share/classes/jdk/jfr/events/TLSHandshakeEvent.java`](https://github.com/openjdk/jdk/pull/31371/changes#diff-6fcb6ccf38f77d1a943ad82829a7b70a3d28ace92e895897963a474222675ed4) Fixed in https://github.com/openjdk/jdk/commit/09283ecd827fe2bc8da5236814c838e5faf9b015 ------------- PR Review Comment: https://git.openjdk.org/jdk/pull/31371#discussion_r3364047058
