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

[Alexandre Julliard (＠julliard)]

Alexandre Julliard (@julliard) commented about tools/widl/metadata.c:
> +};
> -
>
> +static IMAGE_SECTION_HEADER section_header =
> +{
> - .Name = ".text",
> - { .VirtualSize = 0 },
> - .VirtualAddress = 0,
> - .SizeOfRawData = 0,
> - .PointerToRawData = 0,
> - .PointerToRelocations = 0,
> - .PointerToLinenumbers = 0,
> - .NumberOfRelocations = 0,
> - .NumberOfLinenumbers = 0,
> - .Characteristics = IMAGE_SCN_CNT_CODE | IMAGE_SCN_MEM_READ
>
> +};
> -

If you are going to use designated initializers, it doesn't seem necessary to list all the zero fields. Particularly for `.DataDirectory`, it would be a lot more readable as `[IMAGE_DIRECTORY_ENTRY_COM_DESCRIPTOR] = { ... }`.