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

[Alexandre Julliard]

Alexandre Goujon <ale.goujon(a)gmail.com> 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.

-- 
Alexandre Julliard
julliard(a)winehq.org