[PATCH] gdiplus/tests: Test for initial properties of adjustable arrow caps

Nikolay Sivov nsivov at codeweavers.com
Mon Dec 19 11:59:30 CST 2016


On 19.12.2016 20:46, Vincent Povirk wrote:
> On Fri, Dec 16, 2016 at 6:21 AM, Nikolay Sivov <nsivov at codeweavers.com> wrote:
>> +    stat = GdipGetCustomLineCapBaseInset((GpCustomLineCap*)cap, &inset);
>> +    ok(stat == Ok, "Unexpected return code, %d\n", stat);
>> +
> It isn't enough to reject the patch, but I'm curious why you don't
> test this value.
>
>
I only tested this briefly, inset depends on width/height ratio. It felt
like one test is not enough,
so I decided to leave it to come back later. That of course means I can
remove this call altogether.



More information about the wine-devel mailing list