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

[Alfred Agrell (＠Alcaro)]

Alfred Agrell (@Alcaro) commented about tools/widl/metadata.c:
> - for (i = 0; i < STREAM_MAX; i++)
> - {
> -
>    if (!streams[i].data_size) continue;
> -
>    write_data( file, streams[i].data, streams[i].data_size );
> - }
>
> +}
> -
>
> +void write_metadata( const statement_list_t *stmts )
> +{
> - static const BYTE pad[FILE_ALIGNMENT];
> - UINT image_size, file_size, i;
> - FILE *file;
> -
> - if (!do_metadata) return;
> -
> - if (!(file = fopen( metadata_name, "w" )))

This too looks problematic on Windows. Better use "wb".