Re: [PATCH v2 0/5] MR8129: widl: Write the table stream.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/metadata.c:
> +struct assembly_row
> +{
> - UINT   hashalgid;
> - USHORT majorversion;
> - USHORT minorversion;
> - USHORT buildnumber;
> - USHORT revisionnumber;
> - UINT   flags;
> - UINT   publickey;
> - UINT   name;
> - UINT   culture;
>
> +};
> -
>
> +static UINT add_assembly_row( UINT name )
> +{
> - struct assembly_row row = { 0x8004, 255, 255, 255, 255, 0x200, 0, name };

There seems to be names for 0x8004 in https://gitlab.winehq.org/mono/mono/blob/main/mono/metadata/tabledefs.h#L21, can we name it? Same thing for the flags although I'm not sure which one it is in mono.

Also, I think it'd be better to use either designated initialization or initialize culture to 0 too, it's a bit misleading otherwise.