Re: [PATCH 1/4] gdiplus/metafile: Support hatch brushes in playback

[Nikolay Sivov]

On 01.11.2017 19:46, Vincent Povirk wrote:
> +        status =
> GdipCreateHatchBrush(data->BrushData.hatch.HatchStyle, *(ARGB
> *)&data->BrushData.hatch.ForeColor,
> +            *(ARGB *)&data->BrushData.hatch.BackColor, (GpHatch **)brush);
>
> I guess this is OK for this patch, but maybe we should modify the
> definition of EmfPlusARGB to avoid needing things like this?

Sure. You think it's better to have a single ARGB field, or a union to
make components accessible too?