Re: headers, two tests

[Dmitry Timoshkov]

This time it's just the nit picks, I see no serious problems.

"Evan Stade"  wrote:
> +void WINGDIPAPI GdipFree(void* ptr)
> +{
> - if(ptr)
> -
>    HeapFree(GetProcessHeap(), 0, ptr);
>
> +}

There is no need to check a pointer for NULL before HeapFree call, there is
even a janitorial project in Wine to remove such checks.
> +static void test_constructor_destructor(void)
> +{
> - GpStatus status;
> - GpGraphics *pen;
> -
> - status = GdipCreatePen1((ARGB)0xffff00ff, 10.0f, UnitPixel, &pen);

GdipCreatePen1 takes a pointer to GpPen in the last parameter, not GpGraphics.
> - expect(Ok, status);
> - ok(pen, "Expected pen to be initialized");

If you are testing returned pen pointer for NULL, you need to initialize it
to NULL before the call.