Re: [PATCH 1/2] gdi32/tests: Don't treat a return value of COMPLEXREGION from IntersectClipRect() as broken.

[Dmitry Timoshkov]

Zebediah Figura  wrote:
>  ret = IntersectClipRect(hdc, 0, 0, 50, 50);
> - if (ret == COMPLEXREGION)
> - {
> -
>    /* XP returns COMPLEXREGION although dump_region reports only 1 rect */
> -
>    trace("Windows BUG: IntersectClipRect returned %d instead of SIMPLEREGION\n", ret);
> -
>    /* let's make sure that it's a simple region */
> -
>    ret = GetClipRgn(hdc, hrgn);
> -
>    ok(ret == 1, "GetClipRgn returned %d instead of 1\n", ret);
> -
>    dump_region(hrgn);
> - }
> - else
> -
>    ok(ret == SIMPLEREGION, "IntersectClipRect returned %d instead of SIMPLEREGION\n", ret);
> - todo_wine ok(ret == COMPLEXREGION, "wrong region type %d\n", ret);
> - ret = GetClipRgn(hdc, hrgn);
> - ok(ret == 1, "GetClipRgn returned %d instead of 1\n", ret);
> - ret = GetRegionData(hrgn, sizeof(buffer), rgndata);
> - ok(ret == sizeof(RGNDATAHEADER) + sizeof(RECT), "got %u\n", ret);
> - ok(rgndata->rdh.nCount == 1, "got %u rectangles\n", rgndata->rdh.nCount);

Even if every Windows version returns broken result doesn't make it
legitimate to return COMPLEXREGION when the region contains only 1
rectangle. This is clearly broken.