Re: [PATCH 2/5] include/ddk: Add DVD_DESCRIPTOR_HEADER descriptor structure (try 2)

[Alexandre Julliard]

Alexandre Goujon  writes:
> @@ -113,7 +113,16 @@ typedef struct DVD_READ_STRUCTURE {
>      /* From 0 to 4 */
>      UCHAR LayerNumber;
>
> -}DVD_READ_STRUCTURE, * PDVD_READ_STRUCTURE;
> +} DVD_READ_STRUCTURE, *PDVD_READ_STRUCTURE;
> -
>
> +typedef struct _DVD_DESCRIPTOR_HEADER {
> - USHORT Length;
> - UCHAR Reserved[2];
>
> +#if !defined(_midl)
> - UCHAR Data[0];
>
> +#endif
> +} DVD_DESCRIPTOR_HEADER, *PDVD_DESCRIPTOR_HEADER;
> +C_ASSERT(sizeof(DVD_DESCRIPTOR_HEADER) == 4);

Zero-size arrays are not portable and should be avoided.