[TLS] Inconsistent extension definitions

[Yishuai Li <[e-mail]>]

Dear TLS working group,

Here’s a duplicate of GitHub issue tlswg/tls13-rfc#21 I opened today,
which somehow disappeared:

Supported Versions are defined as Variants:

    struct {
        select (Handshake.msg_type) {
            case client_hello:
                 ProtocolVersion versions<2..254>;

            case server_hello: /* and HelloRetryRequest */
                 ProtocolVersion selected_version;
        };
    } SupportedVersions;

while Key Share is defined as separate Constructed Types:

    struct {
        KeyShareEntry client_shares<0..2^16-1>;
    } KeyShareClientHello;

    struct {
        NamedGroup selected_group;
    } KeyShareHelloRetryRequest;

    struct {
        KeyShareEntry server_share;
    } KeyShareServerHello;

Is there a specific reason for choosing different definition styles?
Is it worth unifying them?

Also, is this mailing list the right place for such questions?

Thanks,
Yishuai Li