Re: [v2 PATCH 1/3] gdiplus: Implement GdipGetPenTransform

[Nikolay Sivov]

On Sunday, 31 January 2016, Alistair Leslie-Hughes <
[e-mail]> wrote:
> Signed-off-by: Alistair Leslie-Hughes <[e-mail]
> javascript:;>
>
> dlls/gdiplus/gdiplus_private.h | 1 +
>  dlls/gdiplus/pen.c             | 9 ++++-----
>  dlls/gdiplus/tests/pen.c       | 6 +++---
>  3 files changed, 8 insertions(+), 8 deletions(-)
>
> diff --git a/dlls/gdiplus/gdiplus_private.h
> b/dlls/gdiplus/gdiplus_private.h
> index 1af0bd0..fd45a49 100644
> --- a/dlls/gdiplus/gdiplus_private.h
> +++ b/dlls/gdiplus/gdiplus_private.h
> @@ -197,6 +197,7 @@ struct GpPen{
>      REAL offset;    /* dash offset */
>      GpBrush *brush;
>      GpPenAlignment align;
> - GpMatrix *transform;
>
> };

Why does it have to be a separate allocation? It looks like GpMatrix field
will work fine.