Re: [PATCH v2 0/5] MR8037: widl: Initial support for generating Windows Runtime metadata.

[Rémi Bernon (＠rbernon)]

Rémi Bernon (@rbernon) commented about tools/widl/metadata.c:
> static void build_table_stream( const statement_list_t *stmts )
>  {
> -
>
> static const GUID guid = { 0x9ddc04c6, 0x04ca, 0x04cc, { 0x52, 0x85, 0x4b, 0x50, 0xb2, 0x60, 0x1d, 0xa8 } };
> static const USHORT space = 0x20;
>
> add_string( "" );
> add_userstring( NULL, 0 );
> add_userstring( &space, sizeof(space) );
> add_blob( NULL, 0 );
> -
>
> add_guid( &guid );

Fwiw, not a big issue that early but I gave monodis a try, and it expects a GUID entry first and asserts when missing, then some "#~" data, and segfaults when missing. I'm not sure that should determine the order of priority we should output these sections, but is the current code supposed to work, in some minimal way, with another tool?